website/integrations: add three more policy-expressions to discord-docs#5760
website/integrations: add three more policy-expressions to discord-docs#5760tanberry merged 5 commits intogoauthentik:mainfrom Keyinator:main
Conversation
…ps during enrollment or authentication - Add policy to store oauth-info and create an authentik-avatar-attribute during enrollment or authentication
✅ Deploy Preview for authentik ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5760 +/- ##
==========================================
- Coverage 92.65% 92.63% -0.02%
==========================================
Files 708 708
Lines 34619 34619
==========================================
- Hits 32077 32071 -6
- Misses 2542 2548 +6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
Hi @Keyinator thanks for this PR. It is marked as a draft... are you ready for it to be reviewed and hopefully merged? |
|
Hey @Keyinator thanks for the PR, do you want to make any other changes before we review or should we change a couple of final things ourselves and merge it? |
Sorry for the late reply, I did not notice that the pr was awaiting an answer. I will have another look at it over the weekend and see if there's any more details that can be improved. |
- Fixed issue with wrong return during provider-check - Lint using black Signed-off-by: Keyinator <k3yinator@gmail.com>
✅ Deploy Preview for authentik-storybook canceled.
|
✅ Deploy Preview for authentik-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
tanberry
left a comment
There was a problem hiding this comment.
Thanks @Keyinator for this contribution! A couple of nit-picks about capitalization, and one question, then let's get this merged!
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com> Signed-off-by: Keyinator <k3yinator@gmail.com>
|
All done @tanberry |
|
@Keyinator this PR apparently has merge conflicts... do you want to resolve the and push again, or I can... clone your branch and do it...? I think, lol. |
Should be fixed now |
|
Thank you so much for this contribution and for your perseverance, @Keyinator ! |
* main: (36 commits) ci: use container registry for container build cache (#9809) core: bump lxml from 5.2.1 to 5.2.2 (#9717) web: bump mermaid from 10.9.0 to 10.9.1 in /web (#9734) core: bump scim2-filter-parser from 0.5.0 to 0.5.1 (#9730) web: bump core-js from 3.37.0 to 3.37.1 in /web (#9733) stages/authenticator_webauthn: Update FIDO MDS3 & Passkey aaguid blobs (#9729) translate: Updates for file locale/en/LC_MESSAGES/django.po in zh_CN (#9802) translate: Updates for file locale/en/LC_MESSAGES/django.po in zh-Hans (#9803) core: bump sentry-sdk from 2.2.0 to 2.2.1 (#9807) web: bump the storybook group in /web with 7 updates (#9804) web: bump glob from 10.3.15 to 10.3.16 in /web (#9805) root: docker-compose: remove version top level element (#9631) core, web: update translations (#9790) web: bump API Client version (#9801) web/admin: rework initial wizard pages and add grid layout (#9668) website/integrations: discord: fix typo (#9800) website/integration/netbox: fix group custom pipeline example (#9738) root: add primary-replica db router (#9479) website/integrations: add three more policy-expressions to discord-docs (#5760) website/integrations: netbox: add missing scope configuration (#9491) ...
Changes
New Features
Add two policy-examples for syncing discord roles to authentik groups during enrollment or authentication
Add policy to store oauth-info and create an authentik-avatar-attribute during enrollment or authentication
Checklist
make website)