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

Make some form elements optional for blast database setup #112

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

shreyas-a-s
Copy link
Contributor

@shreyas-a-s shreyas-a-s commented Sep 27, 2024

PR description updated based on issue

Issue #111

This PR makes a number of new settings focused on linkouts optional in the blast database entity form. These settings are not yet well described and thus not clear how to fill out. We will evaluate whether they need to be required at a later date once this module is more stable.

Testing

Create a blast database through the UI and confirm you can leave these fields empty and the page is created without error.

image

@laceysanderson laceysanderson self-requested a review October 2, 2024 17:40
@laceysanderson laceysanderson added Group 2 - User Interface Any issue relating to the user interface overview + submission forms. Group 5 - Blast Database Admin Any issues relating to the admin input of blast databases available to the user. labels Oct 2, 2024
Copy link
Member

@laceysanderson laceysanderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally and this works exactly as expected. Thanks @shreyas-a-s ❤️

@laceysanderson laceysanderson merged commit 87664b8 into tripal:2.x Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Group 2 - User Interface Any issue relating to the user interface overview + submission forms. Group 5 - Blast Database Admin Any issues relating to the admin input of blast databases available to the user.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants