Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: user restriction rework #235

Merged
merged 9 commits into from
Jan 27, 2024
Merged

feat: user restriction rework #235

merged 9 commits into from
Jan 27, 2024

Conversation

or2e
Copy link
Collaborator

@or2e or2e commented Jan 3, 2024

Patch notes:

Webui

  • Blocking reason added. @or2e

    Screenshots

    image


Moderators only

  • For restrictions created now there are 2 fields - public and private reason for blocking. Obviously, the public one will be seen by the user. @or2e

    Screenshots

    image
    image

@or2e or2e linked an issue Jan 3, 2024 that may be closed by this pull request
@or2e or2e changed the title feat: User restriction rework feat: user restriction rework Jan 3, 2024
@or2e or2e requested a review from namidaka January 3, 2024 16:49
@or2e or2e marked this pull request as ready for review January 3, 2024 16:49
Comment on lines 23 to 26
namespace Crpg.Persistence.Migrations
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

Scoped namespace namespace Crpg.Persistence.Migrations;

Copy link
Collaborator

Choose a reason for hiding this comment

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

Still not scoped?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fix fix fix )

@or2e or2e force-pushed the 22-users-note-internal-only branch from 7455457 to d3e835b Compare January 14, 2024 10:32
@or2e or2e requested a review from namidaka January 14, 2024 10:59
@or2e or2e force-pushed the 22-users-note-internal-only branch from c126b60 to 685a422 Compare January 21, 2024 11:57
Comment on lines 23 to 26
namespace Crpg.Persistence.Migrations
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

Still not scoped?

@or2e or2e requested a review from namidaka January 27, 2024 11:30
namidaka
namidaka previously approved these changes Jan 27, 2024
@or2e or2e force-pushed the 22-users-note-internal-only branch from 1a84283 to 8e164c7 Compare January 27, 2024 14:41
@or2e or2e requested a review from namidaka January 27, 2024 14:42
@namidaka namidaka merged commit e703616 into master Jan 27, 2024
4 checks passed
@namidaka namidaka deleted the 22-users-note-internal-only branch January 27, 2024 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

User's note (Internal only)
2 participants