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

Connects to #366. DEA search by protein names. #380

Merged
merged 22 commits into from
Dec 12, 2024

Conversation

jimmyzhen
Copy link
Contributor

Key Changes:

  • Enabled support for DEA results search by protein names
  • Deprecated DEA results search by protein IDs
  • Added support for is_meta flag in searching metabolite DEA results

…search by protein id

Also:
1. Updated primary search instructions on the page
2. Updated function to convert manually input strings to arrays of search terms while keeping the auto-suggestion array as is
3. Updated placeholder text in search input field
4. Fixed bug in which selecting a different radio button didn't clear the manual input string in the search field
5. Removed commented JSX markup that is no longer needed
Also add reducers to support manual pagination controls.
Also add actions to support manual pagination controls.
…ta searches

Also refactor code to support manual pagination controls.
…ntrols

Also:
* Add support for proteomics specific timewise DEA table display
* Delete no longer needed global filter configs
@jimmyzhen jimmyzhen linked an issue Dec 12, 2024 that may be closed by this pull request
@jimmyzhen jimmyzhen requested a review from Copilot December 12, 2024 00:19
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 suggestion.

Files not reviewed (1)
  • src/Search/timewiseTable.jsx: Evaluated as low risk
Comments skipped due to low confidence (5)

src/Search/searchPage.jsx:508

  • The clearInput function should be called within handleRadioChange to ensure the input field is cleared when the radio button changes.
clearInput();

src/Search/searchPage.jsx:134

  • The example for metabolites should be enclosed in double quotes to match the new format.
return 'Example: "amino acids and peptides", "c10:2 carnitine"';

src/Search/searchPage.jsx:166

  • The formatSearchInput function should handle cases where the input string is empty or contains only spaces.
return newArr;

src/Search/searchPage.jsx:254

  • The toggleOmics function should be updated to use the correct parameter names.
onClick={toggleOmics.bind(this, 'omics', key)}

src/Search/searchReducer.js:17

  • The variable 'keys' was changed from a string to an array. Ensure this change is consistently reflected throughout the codebase.
keys: [],

src/Search/searchPage.jsx Outdated Show resolved Hide resolved
Copy link
Collaborator

@jyu-sm jyu-sm left a comment

Choose a reason for hiding this comment

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

tested on the dev instance.

@jimmyzhen jimmyzhen merged commit ee75bbc into dev Dec 12, 2024
6 checks passed
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.

Search enhancement - search by protein names
2 participants