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

fix(KFLUXUI-194): update usage of build trigger #24

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

CryptoRodeo
Copy link
Contributor

Add a tooltip for build trigger details page.
Remove two columns from the component activity page.

Backport of openshift/hac-dev#993

@CryptoRodeo CryptoRodeo changed the title fix(KFLUXBUGS-603): update usage of build trigger fix(KFLUXUI-194): update usage of build trigger Nov 14, 2024
@sahil143
Copy link
Collaborator

/retest

@codecov-commenter
Copy link

codecov-commenter commented Nov 18, 2024

Codecov Report

Attention: Patch coverage is 94.11765% with 1 line in your changes missing coverage. Please review.

Project coverage is 79.76%. Comparing base (8041520) to head (dc8329e).
Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
...s/ComponentDetails/tabs/ComponentBuildSettings.tsx 93.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
- Coverage   79.78%   79.76%   -0.03%     
==========================================
  Files         568      567       -1     
  Lines       21261    21228      -33     
  Branches     5017     5264     +247     
==========================================
- Hits        16963    16932      -31     
+ Misses       4274     4271       -3     
- Partials       24       25       +1     
Flag Coverage Δ
unittests 79.76% <94.11%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@gbenhaim
Copy link
Member

/ok-to-test

@CryptoRodeo
Copy link
Contributor Author

After taking a look at the lines from Codecov, I believe that code coverage can be ignored here.

@@ -30,14 +27,12 @@ export const getContainerImageLink = (url: string) => {

const ComponentsListRow: React.FC<
RowFunctionArgs<ComponentWithLatestBuildPipeline, PacStatesForComponents>
> = ({ obj: component, customData }) => {
> = ({ obj: component }) => {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we do not need that customData, Please remove the data passed to customData in the ComponentListView Table component.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a tooltip for build trigger details page.
Remove two columns from the component activity page.
@sahil143 sahil143 merged commit e5a978b into konflux-ci:main Dec 4, 2024
3 of 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.

4 participants