Skip to content

stages/authenticator_email: Email Authenticator Stage Documentation#12853

Merged
tanberry merged 12 commits intogoauthentik:mainfrom
melizeche:docs/stage_authenticator_email
Feb 24, 2025
Merged

stages/authenticator_email: Email Authenticator Stage Documentation#12853
tanberry merged 12 commits intogoauthentik:mainfrom
melizeche:docs/stage_authenticator_email

Conversation

@melizeche
Copy link
Member

@melizeche melizeche commented Jan 27, 2025 •
edited
Loading

Details

Documentation for Email Authenticator Stage

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

  • The documentation has been updated
  • The documentation has been formatted (make website)

@melizeche melizeche requested a review from a team as a code owner January 27, 2025 20:29
@netlify
Copy link

netlify bot commented Jan 27, 2025 •
edited
Loading

✅ Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit dfa44de
🔍 Latest deploy log https://app.netlify.com/sites/authentik-storybook/deploys/67b8efb75fa4790008bbe499

@netlify
Copy link

netlify bot commented Jan 27, 2025 •
edited
Loading

✅ Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit dfa44de
🔍 Latest deploy log https://app.netlify.com/sites/authentik-docs/deploys/67b8efb7933aad0008e07726
😎 Deploy Preview https://deploy-preview-12853--authentik-docs.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 site configuration.

@melizeche melizeche marked this pull request as draft January 27, 2025 20:33
@melizeche @tanberry
Apply suggestions from code review
a0f7da9 
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com>
Signed-off-by: Marcelo Elizeche Landó <marce@melizeche.com>
@melizeche melizeche requested a review from tanberry January 28, 2025 02:08
@melizeche melizeche mentioned this pull request Jan 28, 2025
Merged
6 tasks
@tanberry tanberry added the deploy_me Deploy the PR changes on a test environment label Jan 28, 2025
@melizeche @tanberry
Apply suggestions from code review
49bc373 
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com>
Signed-off-by: Marcelo Elizeche Landó <marce@melizeche.com>
@codecov
Copy link

codecov bot commented Jan 29, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.64%. Comparing base (4af415f) to head (dfa44de).
Report is 18 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main   #12853      +/-   ##
==========================================
+ Coverage   92.49%   92.64%   +0.15%     
==========================================
  Files         792      792              
  Lines       40231    40231              
==========================================
+ Hits        37211    37274      +63     
+ Misses       3020     2957      -63
Flag Coverage Δ
e2e 48.05% <ø> (+0.51%) ⬆️
integration 24.13% <ø> (ø)
unit 90.43% <ø> (+<0.01%) ⬆️

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 removed the deploy_me Deploy the PR changes on a test environment label Jan 30, 2025
@melizeche melizeche marked this pull request as ready for review February 17, 2025 18:52
@melizeche melizeche requested a review from tanberry February 17, 2025 18:58
tanberry
tanberry approved these changes Feb 17, 2025
View reviewed changes
Copy link
Contributor

@tanberry tanberry left a comment

Choose a reason for hiding this comment

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

Thanks for doing these docs @melizeche ! Ready for the release next week.

@tanberry tanberry merged commit 77c913b into goauthentik:main Feb 24, 2025
86 checks passed
@rissson
Copy link
Member

rissson commented Feb 24, 2025

/cherry-pick version-2025.2

@gcp-cherry-pick-bot
Copy link
Contributor

gcp-cherry-pick-bot bot commented Feb 24, 2025

Cherry-pick failed with Merge error 77c913bfd35c5de9f737eae50a2c6cc344a3659b into temp-cherry-pick-1013de-version-2025.2

rissson pushed a commit that referenced this pull request Feb 24, 2025
@melizeche @tanberry @rissson
stages/authenticator_email: Email Authenticator Stage Documentation (#…
b26a55a 
…12853)

* stages/authenticator_email: Add initial documentation for Email Authenticator Stage

* fix linting/styling

* Apply suggestions from code review

Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com>
Signed-off-by: Marcelo Elizeche Landó <marce@melizeche.com>

* Apply suggestions from code review

Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com>
Signed-off-by: Marcelo Elizeche Landó <marce@melizeche.com>

* rearranged to match the UI

* fix my broken links

* Tweak Emil Authenticator stage, also capitalize names of other stages

* final tweaks

* Add authenticator_validate for Email Authenticator in authenticator_validate documentation

* add version badge

* Fix broken links (.md -> .mdx)

---------

Signed-off-by: Marcelo Elizeche Landó <marce@melizeche.com>
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com>
Co-authored-by: Tana M Berry <tana@goauthentik.com>
@rissson rissson mentioned this pull request Feb 24, 2025
Merged
6 tasks
rissson added a commit that referenced this pull request Feb 24, 2025
@rissson @tanberry @melizeche
stages/authenticator_email: Email Authenticator Stage Documentation (…
2ba7962 
…cherry-pick #12853) (#13218)

Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com>
Co-authored-by: Tana M Berry <tana@goauthentik.com>
Co-authored-by: Marcelo Elizeche Landó <marcelo@goauthentik.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tanberry tanberry tanberry approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@melizeche @rissson @tanberry