-
Notifications
You must be signed in to change notification settings - Fork 29
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
CUMULUS-3987: ElasticSearch Full Removal - Dashboard Release Ticket #1172
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* init PR * fix npm audit failures
* init PR * remove security key for fixture * add fakePassword for password placeholder * add userName for launchpad user placeholder * update changelog * add format to changelog
* first commit * adding tooltip * adding tooltip * PR feedback * PR feedback * allowlisting http-proxy-middleware * PR feedback * removing http-proxy-middleware from allowlist
…when running in Bamboo CI. Rich F. and I were unable to get this test script to run. We continually got 'no tests to run' error.
…hboard into rf/CUMULUS-3861
…est script. Instead, moved that summary information to the Changelog and Readme files.
…’t find any files to test' error when running that particular test script. Also updated the changelog to reflect the new converted test script.
…ssertions and all pass now.
…he one test script: execution.events.js being sure to convert the test code to RTL from Enzyme.
…est scripts that were converted.
This reverts commit 0a251b8.
Jk/cumulus 3861 suggestions
RF/CUMULUS-3861: Upgrade to React 18 and to React Testing Library
* Updated README to make copyable commands usable without editing * Improve CONTRIBUTING to ease onboarding of new contributors: Fixes #1161 Added a section for installing and testing for beginners, outlining what should be done before development begins. * Updated change log * First round of PR required fixes --------- Co-authored-by: doug-newman-nasa <douglas.j.newamn@nasa.gov> Co-authored-by: Doug Newman <douglas.j.newman@nasa.gov>
* CUMULUS-3990: Upgrade localstack from 3.0.0 to 4.0.3 * fix intermittent failure
…upgrade version dependency issues that arise using npm during install. (#1165) * CUMULUS-3862 - updated install instructions * Updated the beginners file as well * Just one more line removal
* Version bump for next Cumulus API release-es phase2 * Version bump for api v20.0.0 * upgrade cypress to 9.7.0 * fix permission issue * change volume
…1171) * first release PR * updating packagelock * removing line from CHANGELOG
jennyhliu
approved these changes
Feb 19, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do not create a squash merge, but use a merge commit.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary: Summary of changes
Addresses CUMULUS-XX: Develop amazing new feature
Changes
PR Checklist