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

Implement the TextDefinitionReplacement ontology change request type #384

Merged
merged 6 commits into from
Jan 28, 2025

Conversation

jvendetti
Copy link
Member

This pull request introduces several enhancements and improvements to the change request process. It adds support for the TextDefinitionReplacement change request type, enabling users to request modifications to the definitions of ontology classes.

Additionally, it improves error handling by addressing cases where users provide an invalid GitHub repository name or attempt to use a repository with the issue tracker disabled.

The code responsible for hiding change request forms on Cancel and Submit actions has been refactored to be more generic and reusable, ensuring consistency across all change request forms, regardless of type.

Changes to the concepts/_details partial were intentionally kept minimal, with the knowledge that this area of the application is undergoing a redesign.

Resolves #381

@jvendetti jvendetti self-assigned this Jan 25, 2025
%div{id: 'editDefinitionFormDiv', 'data-change-requests-target': 'proposalForm'}
- else
-# Note: get_link_for_cls_ajax is in application_helper.rb, it calls auto_link when necessary.
- ajax_links = values.map {|v| get_link_for_cls_ajax(v, @ontology.acronym, '_blank') }

Check notice

Code scanning / Brakeman

Unescaped parameter value. Note

Unescaped parameter value.
@jvendetti jvendetti marked this pull request as ready for review January 28, 2025 18:23
@jvendetti jvendetti merged commit 0d55e1a into master Jan 28, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for "change definition" change request type
1 participant