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

Sections' and Sessions' sort order cannot be influenced #30

Open
MrCsabaToth opened this issue Oct 1, 2018 · 4 comments
Open

Sections' and Sessions' sort order cannot be influenced #30

MrCsabaToth opened this issue Oct 1, 2018 · 4 comments

Comments

@MrCsabaToth
Copy link
Contributor

The Sections and Sessions right now are listed in the order they were entered. There should be a way to change that order.

Expected Behavior

To have some kind of a way to change the order of sessions and sections.

Current Behavior

The Sections and Sessions right now are listed in the order they were entered.

Possible Solution

Either explicitly manually set a rank variable (just like for sponsor levels), or at some later point we can introduce some list widget which allows ordering by drag & drop on the UI?

Steps to Reproduce

  1. Add a new Section. It will be added to the end.
  2. Add a new Session within a Section. It will be added to the end of the Section.

Detailed Description

Normally we'd want to list the sessions grouped by location and sorted by time within that location. The time is currently a free text entry, so it's easier to rely on an explicit rank field, and that will allow more flexibility than an assumed ordering/grouping.

Possible Implementation

Add a rank field to both Section and Session.
Allow the entry and edition of that value, and later maybe even GUI drag & drop order list widget editing at some point in the future.

MrCsabaToth added a commit to MrCsabaToth/DeLorean-v2 that referenced this issue Oct 1, 2018
MrCsabaToth added a commit to MrCsabaToth/DeLorean-v2 that referenced this issue Oct 1, 2018
@MrCsabaToth
Copy link
Contributor Author

MrCsabaToth commented Oct 2, 2018

@neojato
Copy link
Owner

neojato commented Oct 2, 2018

@MrCsabaToth I plan on testing your PR tonight and will merge asap.

@MrCsabaToth
Copy link
Contributor Author

No rush! (the cat suggests otherwise, but no rush)

@MrCsabaToth
Copy link
Contributor Author

PR #31 resolves this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants