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(backoffice): manage artist alley registrations (#160) #178

Merged
merged 12 commits into from
Aug 25, 2024

Conversation

Metawolve
Copy link
Member

This PR adds artist alley management to the backoffice, introducing a new API endpoint for setting the artist alley table registration status, aswell as two new access roles, solving #160 .

It also adds a confirm dialog when trying to delete entities.

Thanks to @maakinoh for building the artist alley management :)

maakinoh and others added 12 commits August 19, 2024 16:57
Artist alley applications can now be accepted or rejected. Also, users may now also filter the entries
Added a dialog for viewing details about an application
Applications can now be deleted by an admin
Detail dialog now also shows a delete button
# Conflicts:
#	src/Eurofurence.App.Domain.Model/ArtistsAlley/TableRegistrationRecord.cs
#	src/Eurofurence.App.Server.Web/Controllers/ArtistsAlleyController.cs
Fixed an error when approving or rejecting an application.
StateChangeRecord was not saved to the database
@Metawolve Metawolve added the feature Request for an entirely new feature label Aug 25, 2024
@Metawolve Metawolve added this to the EF28 (2024) milestone Aug 25, 2024
@Metawolve Metawolve requested review from Rain336 and Fenrikur August 25, 2024 15:15
@Metawolve Metawolve self-assigned this Aug 25, 2024
@Fenrikur Fenrikur changed the title #160 artist alley registrations feat(backoffice): manage artist alley registrations Aug 25, 2024
@Fenrikur Fenrikur changed the title feat(backoffice): manage artist alley registrations feat(backoffice): manage artist alley registrations (#160) Aug 25, 2024
Copy link
Member

@Fenrikur Fenrikur left a comment

Choose a reason for hiding this comment

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

Some minor things still need fixing, but feature looks good in general! Thanks @maakinoh and @Metawolve! ♥
Fixes will happen within our actual repo. (=

@Fenrikur Fenrikur merged commit 129d9df into eurofurence:main Aug 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Request for an entirely new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants