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

fix: Suspended users cannot authenticate#1849

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

Merged
Emyrk merged 18 commits intomainfromstevenmasley/suspended_users
May 31, 2022

Conversation

Emyrk
Copy link
Member

@EmyrkEmyrk commentedMay 27, 2022•
edited
Loading

What this PR does

  • Blocks non 'active' users from authenticating

Screenshot from 2022-05-27 12-10-10

Screenshot from 2022-05-27 13-26-47

TODO:

  • Activate button was removed, so you can only reset the password of a suspended user. Remaining work is to implement that button.

@EmyrkEmyrk marked this pull request as ready for reviewMay 27, 2022 20:57
@EmyrkEmyrk requested a review froma team as acode ownerMay 27, 2022 20:57
@Emyrk
Copy link
MemberAuthor

@vapurrmaid@presleyp Currently clicking "activate" does nothing. No indicator or anything. I am guessing I should get that to work in this PR... ? Would be nice to kick the can on that, as this is the BE work. I can hide the option for now I guess?

@tjcrantjcran added this to theCommunity MVP milestoneMay 30, 2022
- Remove "activate" button
@EmyrkEmyrk merged commit26a2a16 intomainMay 31, 2022
@EmyrkEmyrk deleted the stevenmasley/suspended_users branchMay 31, 2022 13:06
kylecarbs pushed a commit that referenced this pull requestJun 10, 2022
* fix: Suspended users cannot authenticate- Merge roles and apikey extract httpmw- Add member account to make dev- feat: UI Shows suspended error logging into suspended account- change 'active' route to 'activate'
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@kylecarbskylecarbskylecarbs left review comments

@deansheatherdeansheatherdeansheather approved these changes

@greyscaledgreyscaledAwaiting requested review from greyscaled

Assignees

@EmyrkEmyrk

Labels
None yet
Projects
None yet
Milestone
Community MVP
Development

Successfully merging this pull request may close these issues.

5 participants
@Emyrk@kylecarbs@deansheather@misskniss@tjcran
[8]ページ先頭
©2009-2025 Movatter.jp