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

[T.B.R.] Deleting/hiding mensas #83

Open
SubSide opened this issue Oct 8, 2022 · 2 comments
Open

[T.B.R.] Deleting/hiding mensas #83

SubSide opened this issue Oct 8, 2022 · 2 comments
Milestone

Comments

@SubSide
Copy link
Member

SubSide commented Oct 8, 2022

As an admin I want to be able to hide/delete mensas so they are not visible for normal users.

Acc 1: The API should not return hidden mensas when the user is not authorized

I don't like hard deletes. Maybe a visibility flag? Or will we just use the soft delete feature of Laravel?
And will we allow ways to hard delete?

@SubSide SubSide added this to the MVP milestone Oct 8, 2022
@Kipjr
Copy link
Member

Kipjr commented Oct 31, 2022

Use the built-in method of soft delete. Hard delete does not look relevant to me in any case

@SubSide
Copy link
Member Author

SubSide commented Oct 31, 2022

Use the built-in method of soft delete. Hard delete does not look relevant to me in any case

Well, we want the option for admins to soft-delete mensas so they're not visible anymore for the normal user. But like with the previous mensa, there was also a cancelled state, how do we handle that with soft-delete? Or do we see that as flag?

@SubSide SubSide changed the title Deleting/hiding mensas [T.B.R.] Deleting/hiding mensas Oct 31, 2022
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

No branches or pull requests

2 participants