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

v1.2.2 - Small features and bug fixes #704

Closed
wants to merge 6 commits into from
Closed

v1.2.2 - Small features and bug fixes #704

wants to merge 6 commits into from

Conversation

downiec
Copy link
Collaborator

@downiec downiec commented Jan 8, 2025

Fixes: #682 #683 #699 #700 #713

  • Local Pre-commit Checks
  • CI/CD Build

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • If applicable - I have commented my code, particularly in hard-to-understand areas
  • If applicable - I have made corresponding changes to the documentation
  • If applicable - I have added tests that prove my fix is effective or that my feature works
  • If applicable - New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

…changes that will be included in this update)
@downiec downiec self-assigned this Jan 8, 2025
@downiec downiec changed the title Small features and bug fixes v1.2.2 - Small features and bug fixes Jan 8, 2025
downiec and others added 5 commits January 9, 2025 14:30
* Updated documentation, specifically references to the configuration parameters which have changed over time, to use the latest variable names currently used in configuration. Updated requirements to use the latest readthedocs them of Sphinx

* Update version number, add version changelog (to be updated with the changes that will be included in this update)
…inting errors. Fixed linting errors for the initial_projects_data update. Fixed the status tooltip to correctly display the node's status. Updated test fixtures to accurately reflect the node status response (the node names included the https:// in front and thats why status tooltip wasn't finding them. (#714)
… collections popup (#710)

* Added alert for the case that a collection search fails in the collection management popup. Added a test to bring the coverage up to 100% for the search table component. Removed some unused code, the get endpoint function is not used.

* Minor linting error fixes.

* Fixed some backend tests based on the removed/unused functions

* Small fix to test.
@downiec downiec marked this pull request as ready for review January 16, 2025 01:31
@downiec downiec requested a review from sashakames January 16, 2025 01:31
@downiec downiec closed this Jan 16, 2025
@downiec downiec deleted the v1.2.2 branch January 16, 2025 20:06
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.

REACT_APP_ESGF_NODE_URL
1 participant