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.

Sign up for GitHub

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

Jump to bottom

Sorting Thesauri #7214

Closed
etj opened this issue Apr 2, 2021 · 0 comments · Fixed by #7225
Closed

Sorting Thesauri #7214

etj opened this issue Apr 2, 2021 · 0 comments · Fixed by #7225
Assignees
@mattiagiupponi
Labels
enhancement
Milestone
3.2

Comments

@etj
Copy link
Contributor

etj commented Apr 2, 2021

Is your feature request related to a problem? Please describe.

Since GNIP 82 it is possibile to handle multiple thesauri.
Anyway they do not appear in any particolar order (creation order probably).
We want to be able to show them in a given sort order.

Describe the solution you'd like

  • Add a new field for sorting thesaurus at will. Integer, not null, default 0.
  • The few places where the thesauri are selected from the DB should order the selection by the sorting field, and then by id, so to preserve the exisiting ordering whereas this new field is kept at its default value.

Describe alternatives you've considered
N/A

Additional context
N/A
mattiagiupponi added a commit to mattiagiupponi/geonode that referenced this issue Apr 6, 2021
@mattiagiupponi
[Fixes GeoNode#7214] Sorting Thesauri
203f39a
This was referenced Apr 6, 2021
Merged
Merged
@afabiani afabiani added this to the 3.2 milestone Apr 6, 2021
afabiani pushed a commit that referenced this issue Apr 7, 2021
@mattiagiupponi
[Fixes #7214] Sorting Thesauri (#7225)
ebb4153 
Co-authored-by: Alessio Fabiani <[email protected]>
afabiani pushed a commit that referenced this issue Apr 7, 2021
@mattiagiupponi @afabiani
[Fixes #7214] Sorting Thesauri (#7225)
4e4ac1e 
Co-authored-by: Alessio Fabiani <[email protected]>
(cherry picked from commit ebb4153)
afabiani pushed a commit that referenced this issue Apr 7, 2021
@mattiagiupponi
[Fixes #7214] Sorting Thesauri (#7225) (#7246)
5a8e7da 
Co-authored-by: Alessio Fabiani <[email protected]>
(cherry picked from commit ebb4153)

Co-authored-by: mattiagiupponi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mattiagiupponi mattiagiupponi

Labels
enhancement
Projects
None yet
Milestone
3.2
Development

Successfully merging a pull request may close this issue.

[Fixes #7214] Sorting Thesauri mattiagiupponi/geonode
4 participants
@simboss @etj @afabiani @mattiagiupponi