project-zot / zui

UI for zot registry
Apache License 2.0
13 stars 21 forks source link

feat: Implement no data component #377

Closed raulkele closed 1 year ago

raulkele commented 1 year ago

What type of PR is this? feature

Which issue does this PR fix: Closes #371

What does this PR do / Why do we need it: Implement new data component and use it in homepage

If an issue # is not available please add repro steps and logs from IPAMD/CNI showing the issue:

Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades. Has updating a running cluster been tested?:

Does this change require updates to the CNI daemonset config files to work?:

Does this PR introduce any user-facing change?:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

codecov[bot] commented 1 year ago

Codecov Report

Merging #377 (3a0b6bc) into main (087b426) will increase coverage by 0.05%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #377      +/-   ##
==========================================
+ Coverage   87.45%   87.50%   +0.05%     
==========================================
  Files          51       52       +1     
  Lines        1522     1529       +7     
  Branches      397      399       +2     
==========================================
+ Hits         1331     1338       +7     
  Misses        179      179              
  Partials       12       12              
Files Changed Coverage Δ
src/components/Home/Home.jsx 100.00% <100.00%> (ø)
src/components/Shared/NoDataComponent.jsx 100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more