DocHub-ULB / DocHub

A student platform for ULB focused on real student collaboration
https://dochub.be
GNU Affero General Public License v3.0
46 stars 14 forks source link

Use boolean for "staff picks" instead of using the "official" tag #281

Closed Mortinat closed 1 year ago

Mortinat commented 1 year ago

The problem is that not all documents tagged "official" are certified, so it would be better to have a specific boolean field.

C4ptainCrunch commented 1 year ago

Indeed, it seems a good idea to use a dedicated field and not the fact that the "official" tag. If someone wants to learn how to write migrations, this could be a nice first issue as we will have to set the field to True for every document that has the official tag at the moment, then remove the tag.

Mortinat commented 1 year ago

Not sure, it's a good idea to do it like that because official just mean it as been given by the uv/teacher. But certified is not mean to be that, the idea would be to have some moderator that determine which is the best synthesis etc

C4ptainCrunch commented 1 year ago

See https://github.com/DocHub-ULB/DocHub/pull/296 for a more comprehensive discussion.