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

Fix: bug of display list submitted ontologies of account setting #286

Conversation

SirineMhedhbi
Copy link

@SirineMhedhbi SirineMhedhbi commented Jul 4, 2023

Context :

see : #270

Changes :

  • Display list of ontologies by acronym ( Ontologies + views of ontologies) in the liste of Submitted ontologies

image

Copy link
Collaborator

@syphax-bouazzouni syphax-bouazzouni left a 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

@SirineMhedhbi SirineMhedhbi changed the title fix-bug-of-display-views-of-ontology-in-the-list-submitted-ontologies-in-account-setting fix-bug-of-display-list-submitted-ontologies-of-account-setting Jul 5, 2023
Copy link
Collaborator

@syphax-bouazzouni syphax-bouazzouni left a 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|
Copy link
Collaborator

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)

Copy link
Author

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

@SirineMhedhbi SirineMhedhbi changed the title fix-bug-of-display-list-submitted-ontologies-of-account-setting Fix: bug of display list submitted ontologies of account setting Jul 6, 2023
@syphax-bouazzouni syphax-bouazzouni merged commit d86eaff into development Jul 7, 2023
Bilelkihal pushed a commit that referenced this pull request Aug 1, 2023
* fix display list of views of ontologies in Submitted ontologies #270
syphax-bouazzouni added a commit that referenced this pull request Sep 5, 2023
* fix display list of views of ontologies in Submitted ontologies #270
syphax-bouazzouni added a commit that referenced this pull request Sep 6, 2023
… 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 <31127782+SirineMhedhbi@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants