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

feat: Add components selector for commits tab (#857)(#858) #2454

Merged
merged 7 commits into from
Dec 13, 2023
Merged

Conversation

JerrySentry
Copy link
Contributor

@JerrySentry JerrySentry commented Dec 12, 2023

Description

Add a multi selector for filtering by components in the commits page and add filtering functionality to the impacted files commits page.

Screen.Recording.2023-12-13.at.2.28.44.PM.mov

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. In 2022 this entity acquired Codecov and as result Sentry is going to need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

Copy link

netlify bot commented Dec 12, 2023

Deploy Preview for gazebo-staging ready!

Name Link
🔨 Latest commit c17ff74
🔍 Latest deploy log https://app.netlify.com/sites/gazebo-staging/deploys/657a229776809600098c4d52
😎 Deploy Preview https://deploy-preview-2454--gazebo-staging.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

codecov-public-qa bot commented Dec 13, 2023

Codecov Report

Merging #2454 (c17ff74) into main (1c2046c) will decrease coverage by 0.01%.
The diff coverage is 96.87%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2454      +/-   ##
==========================================
- Coverage   98.14%   98.13%   -0.01%     
==========================================
  Files         776      777       +1     
  Lines        9962     9994      +32     
  Branches     2539     2550      +11     
==========================================
+ Hits         9777     9808      +31     
- Misses        183      184       +1     
  Partials        2        2              
Files Coverage Δ
...subRoute/ComponentsSelector/ComponentsSelector.tsx 100.00% <100.00%> (ø)
...ChangedTab/FilesChangedTable/FilesChangedTable.jsx 100.00% <100.00%> (ø)
src/services/commit/useCommit.tsx 98.03% <ø> (ø)
src/services/commit/useCommitComponents.tsx 100.00% <ø> (ø)
...lPage/subRoute/FilesChangedTab/FilesChangedTab.tsx 94.44% <66.66%> (-5.56%) ⬇️
Components Coverage Δ
Assets 59.37% <ø> (ø)
Layouts 97.30% <ø> (ø)
Pages 98.99% <96.87%> (-0.02%) ⬇️
Services 99.44% <ø> (ø)
Shared 99.82% <ø> (ø)
UI 94.33% <ø> (ø)

Continue to review full report in Codecov by Sentry.

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

@codecov-qa
Copy link

codecov-qa bot commented Dec 13, 2023

Codecov Report

Merging #2454 (c17ff74) into main (1c2046c) will decrease coverage by 0.01%.
The diff coverage is 96.87%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2454      +/-   ##
==========================================
- Coverage   98.14%   98.13%   -0.01%     
==========================================
  Files         776      777       +1     
  Lines        9962     9994      +32     
  Branches     2544     2550       +6     
==========================================
+ Hits         9777     9808      +31     
- Misses        183      184       +1     
  Partials        2        2              
Files Coverage Δ
...subRoute/ComponentsSelector/ComponentsSelector.tsx 100.00% <100.00%> (ø)
...ChangedTab/FilesChangedTable/FilesChangedTable.jsx 100.00% <100.00%> (ø)
src/services/commit/useCommit.tsx 98.03% <ø> (ø)
src/services/commit/useCommitComponents.tsx 100.00% <ø> (ø)
...lPage/subRoute/FilesChangedTab/FilesChangedTab.tsx 94.44% <66.66%> (-5.56%) ⬇️
Components Coverage Δ
Assets 59.37% <ø> (ø)
Layouts 97.30% <ø> (ø)
Pages 98.99% <96.87%> (-0.02%) ⬇️
Services 99.44% <ø> (ø)
Shared 99.82% <ø> (ø)
UI 94.33% <ø> (ø)

Continue to review full report in Codecov by Sentry.

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

Copy link

codecov bot commented Dec 13, 2023

Codecov Report

Merging #2454 (c17ff74) into main (1c2046c) will not change coverage.
The diff coverage is 96.87%.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #2454   +/-   ##
=====================================
  Coverage   98.14   98.14           
=====================================
  Files        776     777    +1     
  Lines       9962    9994   +32     
  Branches    2493    2550   +57     
=====================================
+ Hits        9777    9808   +31     
- Misses       183     184    +1     
  Partials       2       2           
Files Coverage Δ
...subRoute/ComponentsSelector/ComponentsSelector.tsx 100.00% <100.00%> (ø)
...ChangedTab/FilesChangedTable/FilesChangedTable.jsx 100.00% <100.00%> (ø)
src/services/commit/useCommit.tsx 98.03% <ø> (ø)
src/services/commit/useCommitComponents.tsx 100.00% <ø> (ø)
...lPage/subRoute/FilesChangedTab/FilesChangedTab.tsx 94.44% <66.66%> (-5.56%) ⬇️
Components Coverage Δ
Assets 59.37% <ø> (ø)
Layouts 97.30% <ø> (ø)
Pages 98.99% <96.87%> (-0.02%) ⬇️
Services 99.44% <ø> (ø)
Shared 99.82% <ø> (ø)
UI 94.33% <ø> (ø)

Continue to review full report in Codecov by Sentry.

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

@JerrySentry JerrySentry changed the title [WIP] feat: Add components selector for commits tab (#857)(#858) feat: Add components selector for commits tab (#857)(#858) Dec 13, 2023
Copy link
Contributor

@rohitvinnakota-codecov rohitvinnakota-codecov left a comment

Choose a reason for hiding this comment

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

I will be fixing up the patch diff in an upcoming change.

@JerrySentry JerrySentry merged commit b56855d into main Dec 13, 2023
41 of 49 checks passed
@JerrySentry JerrySentry deleted the fe_857b branch December 13, 2023 21:38
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.

2 participants