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

[stable] Update build #2851

Open
wants to merge 409 commits into
base: master-stable
Choose a base branch
from
Open

[stable] Update build #2851

wants to merge 409 commits into from

Conversation

github-actions[bot]
Copy link
Contributor

No description provided.

@app-sre-bot
Copy link

Can one of the admins verify this patch?

@codecov-commenter
Copy link

codecov-commenter commented May 29, 2024

Codecov Report

Attention: Patch coverage is 60.68734% with 469 lines in your changes missing coverage. Please review.

Project coverage is 66.16%. Comparing base (79c2b75) to head (9314c52).
Report is 116 commits behind head on master-stable.

Files with missing lines Patch % Lines
src/utils/levenshtein-search.ts 20.12% 120 Missing and 11 partials ⚠️
src/utils/localSearch.ts 20.98% 63 Missing and 1 partial ⚠️
src/components/Search/SearchFeedback.tsx 10.00% 27 Missing ⚠️
src/hooks/useChromeServiceEvents.ts 36.84% 22 Missing and 2 partials ⚠️
src/hooks/useSessionConfig.ts 0.00% 20 Missing and 1 partial ⚠️
src/components/RootApp/ScalprumRoot.tsx 78.48% 17 Missing ⚠️
src/utils/sentry.ts 29.16% 17 Missing ⚠️
src/chrome/create-chrome.ts 44.44% 13 Missing and 2 partials ⚠️
src/state/atoms/localSearchAtom.ts 57.57% 12 Missing and 2 partials ⚠️
src/components/QuickStart/useQuickstartsStates.ts 68.96% 9 Missing ⚠️
... and 44 more
Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                @@
##           master-stable    #2851      +/-   ##
=================================================
+ Coverage          63.27%   66.16%   +2.89%     
=================================================
  Files                195      216      +21     
  Lines               4242     4823     +581     
  Branches             852      961     +109     
=================================================
+ Hits                2684     3191     +507     
- Misses              1530     1588      +58     
- Partials              28       44      +16     
Files with missing lines Coverage Δ
src/@types/types.d.ts 100.00% <100.00%> (ø)
src/auth/OIDCConnector/utils.ts 60.00% <ø> (-0.98%) ⬇️
src/auth/entitlementsApi.ts 63.63% <ø> (ø)
src/auth/rbac.ts 100.00% <ø> (ø)
src/auth/setCookie.ts 100.00% <100.00%> (ø)
src/chrome/update-shared-scope.ts 57.50% <100.00%> (+27.50%) ⬆️
...components/AllServicesDropdown/AllServicesMenu.tsx 87.50% <100.00%> (+0.54%) ⬆️
...ents/AllServicesDropdown/PlatformServicesLinks.tsx 100.00% <100.00%> (ø)
src/components/AppPlaceholder/index.tsx 100.00% <100.00%> (ø)
src/components/BetaSwitcher/index.ts 100.00% <100.00%> (ø)
... and 98 more

... and 17 files with indirect coverage changes

Hyperkid123 and others added 24 commits August 15, 2024 10:02
Add option to disable PF4 styling.
Update Ansible title in Services dropdown
…view-switcher

IQE-3075: Add an OUIA ID for iqe-core to consume
…-5.94.0

🚨 [security] Update webpack 5.92.1 → 5.94.0 (minor)
Allow permission methods on UI module routes.
AAP-29825 add ansible-cloud allowed origin
shannon-donahue and others added 30 commits February 10, 2025 15:03
Remove Satellite from header in itless env
* Updated ITLess flag for apps

* Removed ITLess logic for AllServices page

* Fixed linting errors

* Fixed cypress testing errors

* Fixed cypress testing errors

---------

Co-authored-by: Bryan Florkiewicz <[email protected]>
Fix bug preventing permission error message from appearing
* Updated configs

* Class name updater executed

* Ran codemods, fixed all the errors and relevant warnings

* Fixed build errors

* Fixed design issues

* Removed codemod cache file

* Cleaned up code formatting and replaced incorrect css vars

* Fixed breadcrumbs build issues

* Added dark theme support to the logo

* Fixed ToolbarToggle popper placement to be inline again

* Updated snapshots

* Fixed further design issues

* Updated missed pf-v5 class names

* Replaced remaining v5 css vars with tokens

* Removed local widget layout config

* Updated Navigation snapshot after skeleton color change

* Added redux override and updated redhat-cloud-services/types

* Re-added 100vh styling to AllServicesDropdown backdrop

* Reverted config changes to output build in /stable dir

* Fixed lint errors

* Bumped fec config-utils version

* Removed modification for authing against prod

* Fixed error thrown when BreadcrumbProps is not passed to the Header

* Fixed cypress component tests

* Fixed error when querying root element classes

* Updated selectors in cypress component tests

* Fixed toolbar buttons styling per design feedback

* Updated cypress snapshots

* Fixed styling issues found by design during review

* Remove showdown dependency

* Fix user dropdown menu placement in the DOM

* Fix failing user toggle snapshot test

---------

Co-authored-by: Bryan Florkiewicz <[email protected]>
Update console.dev SSO exception
Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
* Service dropdown updates ( right side)

* lint fixes

* fix fav icon position

* styling tweaks
… digest to 9b18077 (#3019)

Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
Co-authored-by: red-hat-konflux[bot] <126015336+red-hat-konflux[bot]@users.noreply.github.com>
Co-authored-by: Martin Maroši <[email protected]>
Fix dev and ux soo url exception
* working drawer with websocket; needs toggle and ref fix

* Made toggleDrawer() function and passed it notifications drawer

* Moved notifications drawer test to notif-frontend

* WIP module nonsense

* Get unreadnotifications

* Moving stuff out of panelcontent

* Working with scope (and context from module)

* Type on notificationsDrawerContext

* Adding notif scope earlier and removing the context

* Passing Spinner in place of DrawerContent on error

* Removing unnecessary notificationProps from DOM element

* Moving bell to notifications

* Moving hooks to hooks dir and cleaning

* Fixing tools test with scalprum context

* Declaring props differently

---------

Co-authored-by: Blake Holifield <[email protected]>
Co-authored-by: Justin Orringer <[email protected]>
Co-authored-by: Martin Maroši <[email protected]>
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.