search

maevsi / sqitch

maevsi's database migrations.
https://maev.si
1 stars 3 forks source link

feat(report): add report system #61

Closed myyxl closed 1 month ago

myyxl commented 7 months ago

Since we need a report system I added a table for that.

~The policies are still missing and will be implemented soon.~

Do we need a new role for the report system? Since a normal account shouldn't be able to access the reports, right?

dargmuesli commented 7 months ago

Great question: A normal account should very much be able to access report! For creating reports, that is, and monitoring the status of their reports. A policy defining the creator of a report to be the only one able to access it must be defined of course.

How admins work with reports is another topic. We could set up email notifications or build an admin dashboard. Notifications make more sense for now, I'd say.

myyxl commented 7 months ago

Notifications are a solution yes, but we still need to manage these reports, right? So we need some kind of interface. Unless we want to manually fiddle with the database.

dargmuesli commented 7 months ago

Let's fiddle with the database until there are some reports or reports that need to be managed by a non-admin.

maevsi-bot commented 1 month ago

:tada: This PR is included in version 3.5.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: