Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat(auth): Add TOTP support in Project and Tenant config#682

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
pragatimodi wants to merge28 commits intomain
base:main
Choose a base branch
Loading
frommfa-totp

Conversation

@pragatimodi
Copy link
Contributor

@pragatimodipragatimodi commentedMar 15, 2023
edited by lahirumaramba
Loading

TOTP Support for Project and Tenant

RELEASE NOTES: AddedTOTP as a multi-factor option in project and tenant configuration.

hspak reacted with thumbs up emoji
2. Added TOTP Support for Tenant3. Added Unit Tests for MFA Config4. Added Unit Tests for Tenant Config with respect to MFA
2. Unit tests for project_config_mgt3. Formatting changes
@pragatimodipragatimodi marked this pull request as draftMarch 15, 2023 20:32
@pragatimodipragatimodi marked this pull request as ready for reviewMarch 21, 2023 18:35
@lahirumarambalahirumaramba added release-note release:stageStage a release candidate labelsMar 29, 2023
lahirumarambaand others added3 commitsMay 26, 2023 10:59
* `v2beta1` -> `v2`* Reverting auto formatting changes* undo auto formatting
It's useful to be able to navigate to the release notes easilyfrom the package index when upgrading."Release Notes" is a special keyword that will have the scroll iconin the project page.A random example:*https://pypi.org/project/streamlit/*https://github.com/streamlit/streamlit/blob/815a3ea6fa3e7f9099b479e8365bd3a5874ddc35/lib/setup.py#L111Co-authored-by: Lahiru Maramba <llahiru@gmail.com>
@pragatimodipragatimodiforce-pushed themfa-totp branch 2 times, most recently fromcd7aed2 to0684915CompareMay 26, 2023 18:03
Copy link

@Xiaoshouzi-ghXiaoshouzi-gh left a comment

Choose a reason for hiding this comment

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

LGTM with comments.

Importing configs directlyfix warningsfix warningsfix warnings
pragatimodiand others added4 commitsSeptember 27, 2023 13:37
Co-authored-by: Kevin Cheung <kevinthecheung@users.noreply.github.com>
Co-authored-by: Kevin Cheung <kevinthecheung@users.noreply.github.com>
@pstrav
Copy link

@kevinthecheung@Xiaoshouzi-gh do we have an ETA on this? Thanks!

hspak reacted with thumbs up emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@kevinthecheungkevinthecheungkevinthecheung approved these changes

@Xiaoshouzi-ghXiaoshouzi-ghXiaoshouzi-gh approved these changes

@lahirumarambalahirumarambaAwaiting requested review from lahirumaramba

+1 more reviewer

@renkelvinrenkelvinrenkelvin approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

release:stageStage a release candidaterelease-note

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@pragatimodi@pstrav@lahirumaramba@renkelvin@kevinthecheung@Xiaoshouzi-gh@samueldg

[8]ページ先頭

©2009-2025 Movatter.jp