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

Extractor: flattened category hierarchy list needs to be sorted by level1, 2, 3 etc #6668

Closed
davidwatkins73 opened this issue Jun 29, 2023 · 0 comments
Closed

Extractor: flattened category hierarchy list needs to be sorted by level1, 2, 3 etc #6668

davidwatkins73 opened this issue Jun 29, 2023 · 0 comments
Assignees
@davidwatkins73
Labels
fixed (test & close) An issue has been fixed, merged into master and is ready for further testing QoL Quality of Life small change Should be a simple, low complexity issue
Milestone
1.52

Comments

@davidwatkins73
Copy link
Member

Description

Currently it's un-ordered which is inconvenient for users

(Build on #6662 )

Needs some order by clauses added to the dynamic sql generated via jOOQ in this class:

MeasurableCategoryExtractor

Resourcing

We intend to contribute this feature
@davidwatkins73 davidwatkins73 added small change Should be a simple, low complexity issue QoL Quality of Life labels Jun 29, 2023
@davidwatkins73 davidwatkins73 self-assigned this Jun 30, 2023
davidwatkins73 added a commit to davidwatkins73/waltz-dev that referenced this issue Jun 30, 2023
@davidwatkins73
Flattened Measurable Hierarchy extractor should sort by l1,l2,l3 etc
85ac559 
finos#6668
davidwatkins73 added a commit to davidwatkins73/waltz-dev that referenced this issue Jun 30, 2023
@davidwatkins73
Flattened Measurable Hierarchy extractor should sort by l1,l2,l3 etc
f9acb45 
finos#6668
@davidwatkins73 davidwatkins73 mentioned this issue Jun 30, 2023
Merged
davidwatkins73 added a commit to davidwatkins73/waltz-dev that referenced this issue Jun 30, 2023
@davidwatkins73
Flattened Measurable Hierarchy extractor should sort by l1,l2,l3 etc
e57b4c2 
finos#6668
@jessica-woodland-scott-db jessica-woodland-scott-db added the fixed (test & close) An issue has been fixed, merged into master and is ready for further testing label Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@davidwatkins73 davidwatkins73

Labels
fixed (test & close) An issue has been fixed, merged into master and is ready for further testing QoL Quality of Life small change Should be a simple, low complexity issue
Projects
None yet
Milestone
1.52
Development

No branches or pull requests
2 participants
@davidwatkins73 @jessica-woodland-scott-db