-
Notifications
You must be signed in to change notification settings - Fork 95
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
Sort concepts in hierarchical view by label and locale #1626
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
null labels should be handled, otherwise this looks good!
@@ -61,7 +61,7 @@ describe('Vocabulary home page', () => { | |||
cy.get('#tab-alphabetical').contains('a', 'care institutions').click() | |||
|
|||
// check the concept prefLabel | |||
cy.get('#concept-heading h1').invoke('text').should('equal', 'care institutions') | |||
cy.get('#concept-heading h1', {'timeout': 15000}).invoke('text').should('equal', 'care institutions') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this change necessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was failing before without the timeout but now it isn't anymore. I'll take it out
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, please check the SonarCloud suggestions
|
Reasons for creating this PR
Concepts in hierarchical view are not currently sorted. This PR add functionality to sort concepts by their label according to locale.
Link to relevant issue(s), if any
Description of the changes in this PR
Sort concepts in hierarchy by label and locale when they are added to hierarchy tree
Addresses requirement 9 in #1521
Known problems or uncertainties in this PR
Concepts are only sorted by label and locale, not notation codes or notation sorting style
Checklist
.sr-only
class, color contrast)