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

Talk priorities #196

Merged
merged 3 commits into from
Dec 19, 2023
Merged

Talk priorities #196

merged 3 commits into from
Dec 19, 2023

Conversation

r3trofitted
Copy link
Contributor

Since we're starting to add low-priority talks (i.e. talks that are mostly proposed to fill in a short line-up, if need be), having a clear way to point them out could be useful. And while we're at it, why not an the option of marking a talk as "high priority" so that room could be made for it (e.g. for an special guest)?

Note that this attribute is not user-facing – only admins should have the ability to decide that a talk as is low- or high-priority. By default, the priority of a talk is "normal", and this level should be kept most of time.

Since we're starting to add low-priority talks (i.e. talks that are
mostly proposed to fill in a short line-up, if need be), having a
clear wat to point them out could be useful. And while we're at it, why
not an the option of marking a talk as "high priority" so that room
could be made for it (e.g. for an special guest).

Note that this attribute is not user-facing – only admins should have
the ability to decide that a talk as is low- or high-priority. By
default, the priority of a talk is "normal", and this level should be
kept most of time.
I believe that we could do something fancy, like changing the text color
for low- or high-priority talks, instead of, or in addition to, an extra
column in the table, but this should be enough for now.
@adrienpoly
Copy link
Collaborator

Thanks @r3trofitted for this feature.
one suggestion, is there a hint field or equivalent in RailsAdmin that we could use to give the admin some guidelines for this setting?

@r3trofitted
Copy link
Contributor Author

Good point. I haven't thought of that, haven't checked, and know too little about Rails Admin to answer off the top of my head. Let me have a look.

@adrienpoly adrienpoly merged commit eb36083 into parisrb:master Dec 19, 2023
1 check passed
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.

2 participants