WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@atereshkin
Copy link
Contributor

Details

This PR implements missing "connect account" functionality for the Telegram source. When a Telegram source is configured, existing users should be able connect their Telegram account by going to Settings -> Connected services. Currently this functionality is missing and this PR addresses that.

A respective view is added in sources/telegram/api/source.py and renderConnectButton is implemented in SourceSettingsTelegram.ts.

This closes #18467

Checklist

  • Local tests pass (ak test authentik/)
  • The code has been formatted (make lint-fix)

If an API change has been made

  • The API schema has been updated (make gen-build)

If changes to the frontend have been made

  • The code has been formatted (make web)

If applicable

  • [n/a] The documentation has been updated
  • [n/a] The documentation has been formatted (make docs)

@atereshkin atereshkin requested review from a team as code owners December 2, 2025 11:25
@netlify
Copy link

netlify bot commented Dec 2, 2025

Deploy Preview for authentik-docs canceled.

Name Link
🔨 Latest commit 4341fe4
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/692f2955e6b5e00008e8ac55

@netlify
Copy link

netlify bot commented Dec 2, 2025

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit 4341fe4
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/692f295534cb080008b1b977

@netlify
Copy link

netlify bot commented Dec 2, 2025

Deploy Preview for authentik-integrations ready!

Name Link
🔨 Latest commit 4341fe4
🔍 Latest deploy log https://app.netlify.com/projects/authentik-integrations/deploys/692f295534cb080008b1b97b
😎 Deploy Preview https://deploy-preview-18517--authentik-integrations.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@codecov
Copy link

codecov bot commented Dec 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.11%. Comparing base (eb60276) to head (4341fe4).
⚠️ Report is 11 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #18517      +/-   ##
==========================================
+ Coverage   93.02%   93.11%   +0.08%     
==========================================
  Files         911      931      +20     
  Lines       49848    50880    +1032     
==========================================
+ Hits        46373    47375    +1002     
- Misses       3475     3505      +30     
Flag Coverage Δ
e2e 44.84% <50.68%> (-0.41%) ⬇️
integration 22.73% <1.36%> (-0.15%) ⬇️
unit 91.37% <100.00%> (+0.16%) ⬆️
unit-migrate 91.41% <100.00%> (+0.16%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rissson rissson moved this from Todo to Needs review in authentik Core Dec 2, 2025
@rissson rissson added this to the Release 2025.12 milestone Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Needs review

Development

Successfully merging this pull request may close these issues.

No way to connect a Telegram account to an existing user account

3 participants