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

Environment Sync, Beta to Stable #1070

Merged
merged 13 commits into from
Mar 2, 2023
Merged

Environment Sync, Beta to Stable #1070

merged 13 commits into from
Mar 2, 2023

Conversation

cdcabrera
Copy link
Member

@cdcabrera cdcabrera commented Feb 28, 2023

What's included

Notes

Commits

Documentation

Code Refactoring

Chores

cdcabrera and others added 12 commits February 20, 2023 14:04
* config, rhacs, rhel, rhods, rhosak, instance id
* services, replace inventory with instance id
* graphCard, adjust card keys
* graphCardChart, remove minHeight div
* graphCardMetricTotals, replace pf flex, styling, minHeight
* styling, usage-graph adjustments
* authentication, redirect errorRoute ref
* build, spelling, jest, local run nav
* config, generate products, routes
* productView, replaced routerHelpers array ref
* productViewMissing, updated routerHelpers
* redirect, simplify, prep router npm update
* redux, viewReducer, updated routerHelpers
* router, prep router npm update
* routerHelpers, refactor, removed unused funcs
* tests, mock react-router-dom
* authentication, use optin directly
* optinView, add testing attributes
* redirect, remove, use optin directly
* index, remove chrome init mock
* redux, remove chrome init
* services, remove deprecated chrome init
* build, routing package, module ref, spelling
* configs, sorted products, aliases, simplify routes
* locale, OpenShift strings
* AppEntry, remove router component
* app, locale loading checks
* helpers, add memoize, object freeze
* authenticationContext, remove nav history push
* i18n, minor check to avoid reload
* productView, useRouteDetail hook update
* productViewMissing, add useNavigate, RouteDetail hooks
* redux, viewReducer, appTypes for storing route ref
* router, simplify, useSetRouteDetail for config load
* routerContext, useRouteDetail, wrap location, navigate
* routerHelpers, restructure getRouteConfigByPath, memo
* authContext, replace setAppName with updateDocTitle
* redux, platformActions, types, remove methods
* platformServices, remove deprecated methods
* build, docs action linter
* docs, jsdoc generate code readme
@cdcabrera cdcabrera added blocked Prevented from continuing 202304 project phase labels Feb 28, 2023
@cdcabrera cdcabrera removed the blocked Prevented from continuing label Mar 1, 2023
@codecov-commenter
Copy link

codecov-commenter commented Mar 1, 2023

Codecov Report

Merging #1070 (c553c74) into main (d508fbd) will decrease coverage by 0.09%.
The diff coverage is 91.74%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1070      +/-   ##
==========================================
- Coverage   93.56%   93.48%   -0.09%     
==========================================
  Files         122      123       +1     
  Lines        4071     4113      +42     
  Branches     1662     1677      +15     
==========================================
+ Hits         3809     3845      +36     
- Misses        245      251       +6     
  Partials       17       17              
Impacted Files Coverage Δ
src/common/dateHelpers.js 89.83% <ø> (ø)
src/common/downloadHelpers.js 93.33% <ø> (ø)
src/components/bannerMessages/bannerMessages.js 100.00% <ø> (ø)
...components/bannerMessages/bannerMessagesContext.js 83.33% <ø> (ø)
src/components/chart/chart.js 90.90% <ø> (ø)
src/components/chart/chartAxisLabel.js 100.00% <ø> (ø)
src/components/chart/chartContext.js 100.00% <ø> (ø)
src/components/chart/chartElements.js 86.48% <ø> (ø)
src/components/chart/chartHelpers.js 90.36% <ø> (ø)
src/components/chart/chartIcon.js 97.67% <ø> (ø)
... and 116 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d508fbd...c553c74. Read the comment docs.

@cdcabrera
Copy link
Member Author

cdcabrera commented Mar 1, 2023

20230228 - 20230301 env issues, blocking until confirmations come through. Would like to get #1071 in.

@cdcabrera cdcabrera added the blocked Prevented from continuing label Mar 1, 2023
@cdcabrera cdcabrera marked this pull request as ready for review March 2, 2023 02:18
@cdcabrera
Copy link
Member Author

Postponing #1071

@cdcabrera cdcabrera removed the blocked Prevented from continuing label Mar 2, 2023
@cdcabrera cdcabrera merged commit 5d4ef80 into main Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
202304 project phase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants