Skip to content

hotfix(ticket-importer): add tg proxy#7451

Merged
nomerdvadcatpyat merged 5 commits intomainfrom
hotfix/add-proxy
Apr 10, 2026
Merged

hotfix(ticket-importer): add tg proxy#7451
nomerdvadcatpyat merged 5 commits intomainfrom
hotfix/add-proxy

Conversation

@Alexander-Turkin
Copy link
Copy Markdown
Contributor

@Alexander-Turkin Alexander-Turkin commented Apr 10, 2026

Summary by CodeRabbit

  • Chores
    • Updated Helm submodule reference.
    • Updated ticket-importer submodule reference.
    • Updated employee-bot submodule reference.
    • Adjusted referenced subproject revisions; no public APIs or exported interfaces were changed.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 10, 2026

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f391661e-4a13-4965-b4a6-1b6e60c18f1a

📥 Commits

Reviewing files that changed from the base of the PR and between 1cd8bd3 and f6d3bf7.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (3)
  • .helm
  • apps/employee-bot
  • apps/ticket-importer
✅ Files skipped from review due to trivial changes (3)
  • apps/ticket-importer
  • apps/employee-bot
  • .helm

📝 Walkthrough

Walkthrough

Three git submodules had their recorded commit hashes updated: .helm now references e6d6c0ad..., apps/ticket-importer now references 0015b95e..., and apps/employee-bot now references 33bd8b95....

Changes

Cohort / File(s) Summary
Helm submodule
.helm
Updated recorded submodule commit hash from 881da5558961950b055b5d2e0bdc8363fea4ac7f to e6d6c0ad227f3aaf91b95721608de6d42e6e4dc1.
Ticket importer submodule
apps/ticket-importer
Updated recorded submodule commit hash from 982ee5bc37da097f2866582cc19cd24e15fe2a3d to 0015b95e4f97ec261ebb50533f1f473658f89dac.
Employee bot submodule
apps/employee-bot
Updated recorded submodule commit hash from 3f08182615417b7451d05bcf409fb49972a41a2e to 33bd8b95ba1d506541b1b20cce7e6072365a35de.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 I hopped through refs with twitching nose,
Three hashes changed where tidy subtrees grow.
Helm, ticket, bot — a quiet, careful sweep,
New footprints left where old commits sleep. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title claims to add a TG proxy to ticket-importer, but the actual changes update submodule references across three unrelated modules (.helm, ticket-importer, and employee-bot) without any evidence of proxy-related modifications. Update the PR title to accurately reflect that it updates multiple submodule references (e.g., 'chore: update submodule references for .helm, ticket-importer, and employee-bot') or clarify the actual purpose of these submodule updates.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hotfix/add-proxy

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2fdb1a6b57

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

yarn.lock Outdated
express: ^4.17.1
graphql: ^16.10.0
graphql-tag: ^2.12.6
https-proxy-agent: ^7.0.1
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P0 Badge Re-indent added dependencies in yarn.lock

The inserted https-proxy-agent line is not indented under @app/ticket-importer's dependencies map (same issue for node-telegram-bot-api and socks-proxy-agent), which makes the lockfile syntactically invalid. In this commit state, node .yarn/releases/yarn-3.2.2.cjs install --immutable --mode=skip-build fails with YAMLException: bad indentation, so any local or CI Yarn workflow that reads yarn.lock is blocked.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Contributor

@Alllex202 Alllex202 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Disable links preview for messages :

link_preview_options: { is_disabled: true }

Copy link
Copy Markdown
Contributor

@Alllex202 Alllex202 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@sonarqubecloud
Copy link
Copy Markdown

@nomerdvadcatpyat nomerdvadcatpyat merged commit d3fac5e into main Apr 10, 2026
42 of 44 checks passed
@nomerdvadcatpyat nomerdvadcatpyat deleted the hotfix/add-proxy branch April 10, 2026 16:59
Alexander-Turkin added a commit that referenced this pull request Apr 10, 2026
* hotfix(ticket-importer): add tg proxy

* hotfix(ticket-importer): add tg proxy

* fix(condo): add tg proxy

* fix(condo): add tg proxy

* fix(condo): add tg proxy

(cherry picked from commit d3fac5e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants