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

main >> prod-beta #297

Merged
merged 8 commits into from
Dec 14, 2023
Merged

main >> prod-beta #297

merged 8 commits into from
Dec 14, 2023

Conversation

PreetiW
Copy link
Contributor

@PreetiW PreetiW commented Dec 7, 2023

main >> prod-beta 💥

main >> prod-beta

Feel free to remove this section from PR description once done.

Why do we need this change? 💭

Please include the context of this change here.

Documentation requires update? 📝

  • Yes
  • No

Security Checklist 🔒

Upon raising this PR please go through RedHatInsights/secure-coding-checklist

💂‍♂️ Checklist 🎯

  • Bugfix
  • New Feature
  • Refactor
  • Unittests Added
  • DRY code
  • Dependency Added

Additional 📣

Feel free to add any other relevant details such as links, notes, screenshots, here.

r14chandra and others added 6 commits October 26, 2023 16:10
* RHIROS-492 upgrade react-router-dom to v6

* add piplock file and upgrade packages

* fix routes

* add webpack-cli

* use InsightsLink component and useLocation and useParams hooks

* update tests

* remove withRouter HOC

* update FEC package version

* add babel-runtime package

* fix lodash-es error

* add fast-text-encoding polyfill package
* RHIROS-1163 add suggested instance types table with static data

* RHIROS-1222 integrate with /suggested_instance_types endpoint

* handle UI for errors, change action type naming

* use error object
* fix(RHINENG-3653): Update Groups column default text

* replace additional N/A Text
@PreetiW PreetiW requested a review from r14chandra December 7, 2023 11:38
* RHIROS-1164 implement sorting on suggested instance types table

* code refactor
@codecov-commenter
Copy link

codecov-commenter commented Dec 8, 2023

Codecov Report

Attention: 66 lines in your changes are missing coverage. Please review.

Comparison is base (d648faa) 20.54% compared to head (bfc00bd) 20.10%.
Report is 23 commits behind head on prod-beta.

❗ Current head bfc00bd differs from pull request most recent head fd14d68. Consider uploading reports for the commit fd14d68 to get more accurate results

Files Patch % Lines
...nents/InstanceTypes/SuggestedInstanceTypesTable.js 0.00% 28 Missing and 1 partial ⚠️
src/Utilities/api.js 0.00% 8 Missing and 1 partial ⚠️
...uggestedInstanceTypes/RosSuggestedInstanceTypes.js 0.00% 8 Missing ⚠️
src/store/suggestedInstanceTypes.js 0.00% 6 Missing ⚠️
src/Routes.js 0.00% 3 Missing ⚠️
...c/Components/SystemDetail/SystemRecommendations.js 0.00% 2 Missing ⚠️
src/Routes/RosSystemDetail/RosSystemDetail.js 0.00% 2 Missing ⚠️
src/store/entitiesReducer.js 0.00% 2 Missing ⚠️
src/Components/RosTable/RenderColumn.js 0.00% 1 Missing ⚠️
src/Routes/RosPage/RosPage.js 0.00% 1 Missing ⚠️
... and 3 more
Additional details and impacted files
@@              Coverage Diff              @@
##           prod-beta     #297      +/-   ##
=============================================
- Coverage      20.54%   20.10%   -0.45%     
=============================================
  Files             45       47       +2     
  Lines            915      965      +50     
  Branches         170      181      +11     
=============================================
+ Hits             188      194       +6     
- Misses           676      721      +45     
+ Partials          51       50       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

* RHIROS-1302 enable group sorting

* code refactor - moved groups object to constant file
@PreetiW PreetiW merged commit aed2b75 into prod-beta Dec 14, 2023
4 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.

5 participants