-
Notifications
You must be signed in to change notification settings - Fork 5
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
Fix: bug of display list submitted ontologies of account setting #286
Fix: bug of display list submitted ontologies of account setting #286
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.
Write a better PR name
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.
Does not resolve the issue.
See #270 (comment)
@@ -113,6 +113,10 @@ | |||
- @admin_ontologies.each do |ont| | |||
.account-page-submitted-ontology{data: {controller: 'tooltip'}, title: ont.name} | |||
%a{href: "/ontologies/#{ont.acronym}"}= ont.acronym | |||
- unless ont.views.nil? || ont.views.empty? | |||
- ont.views.each do |view| |
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.
This solution is not good.
The goal is not to show the views of the submitted ontologies.
But to show all the submitted ontologies and views of the users (currently only the ontologies are listed)
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.
@SirineMhedhbi remove this code and open a new PR
* fix display list of views of ontologies in Submitted ontologies #270
* fix display list of views of ontologies in Submitted ontologies #270
… and Bug fixes (#335) * Merge pull request #281 from ontoportal-lirmm/fix-bug-in-slices-links fix bug in slices links * Merge pull request #212 from ontoportal-lirmm/fix/signup-fields-validation Fix: Signup page input fields validation * Merge pull request #214 from ontoportal-lirmm/feature/update-account-page-design Feature: Update submited ontologies and created projet sections design * Merge pull request #237 from ontoportal-lirmm/feature/ecoportal/groups-categories-administration Feature: Add categories and groups administration * Merge pull request #273 from ontoportal-lirmm/fix/bug-ANAEETHES-ontology Fix: Ontology description text display when it is a complex object not a string * Merge pull request #277 from ontoportal-lirmm/Fix-alphabetical-sorting Fix: Alphabetical sorting in the the collection members display * Fix: bug of display list submitted ontologies of account setting (#286) * fix display list of views of ontologies in Submitted ontologies #270 * fix bug of display text in alert when there is no collections (#285) fix the alert message text that is wrapping and overflowing out #266 * Merge pull request #287 from ontoportal-lirmm/fix-bug-reset-password Fix: bug of reset password * Fix bug of select list view of ontology (#289) * Add Chosen.js functionality to fix bug of select views of ontolgy * Add Chosen.js functionality for list views ontologies * fix bug of update ontology where there is no view * Refactor code * Delete ponse = @ontology.update * Merge pull request #295 from ontoportal-lirmm/Fix-display-the-list-of-concepts-in-a-collection-at-one-sight Fix: display the list of concepts in a collection at one sight * Merge pull request #291 from ontoportal-lirmm/Fix-remove-views-of-submitted-ontologies-in-account-setting Fix: Remove views ontologies in list of submitted ontologies * Merge pull request #280 from ontoportal-lirmm/Fix-Ontology-style-text-display-when-it-is-uri Fix: ontology style text display when it is uri * Merge pull request #254 from ontoportal-lirmm/feature/add-ontology-count-to-group-and-category Feature: add column ontologies count to categories and-groups admin page * Merge pull request #302 from ontoportal-lirmm/feature/migrate-fair-score-js-code Feature: Migrate fair score JS code to Stimulus controllers * fix generated bug after edit group and category (#334) --------- Co-authored-by: SirineMhedhbi <[email protected]>
Context :
see : #270
Changes :