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

UIDATIMP-1438: Update Log details screen to support multiple holdings & items (Rancher) #1439

Merged
merged 4 commits into from
Jul 13, 2023

Conversation

OleksandrHladchenko1
Copy link
Contributor

Purpose

Approach

Refs

Screenshots

@github-actions
Copy link

github-actions bot commented Jul 13, 2023

Jest Unit Test Statistics

       1 files  ±  0     217 suites  +10   17m 49s ⏱️ - 1m 43s
1 145 tests +28  1 145 ✔️ +28  0 💤 ±0  0 ±0 
1 156 runs  +28  1 156 ✔️ +28  0 💤 ±0  0 ±0 

Results for commit 2a262f8. ± Comparison against base commit b04ad92.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 13, 2023

BigTest Unit Test Statistics

0 tests  ±0   0 ✔️ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ±0 

Results for commit 2a262f8. ± Comparison against base commit b04ad92.

♻️ This comment has been updated with latest results.

@sonarcloud
Copy link

sonarcloud bot commented Jul 13, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 15 Code Smells

99.6% 99.6% Coverage
0.0% 0.0% Duplication

@OleksandrHladchenko1 OleksandrHladchenko1 merged commit f519eb6 into multiples-feature Jul 13, 2023
6 checks passed
OleksandrHladchenko1 added a commit that referenced this pull request Aug 8, 2023
…1447)

* UIDATIMP-1440: Update Log JSON screen to support multiple items (#1438)

* UIDATIMP-1439: Update Log JSON screen to support multiple holdings (Rancher) (#1437)

* UIDATIMP-1438: Update Log details screen to support multiple holdings & items (Rancher) (#1439)

* UIDATIMP-1438: Update Log details screen to support multiple holdings & items

* UIDATIMP-1438: Add PropTypes

* UIDATIMP-1443: Improve mapping profile for Holdings for handling multiple Holdings mapping (#1436)

* multiple-feature: Fix error in records table

* multiple-feature: add no value if no record found

* Fixes for map

* multiple-feature: Adjust RecordsTable component

* Revert "UIDATIMP-1443: Improve mapping profile for Holdings for handling multiple Holdings mapping (#1436)"

This reverts commit e360d19.

* multiple-feature: Add general items error displaying

* multiple-feature: Replace Error with No action

* multiple-feature: refactor code

* multiple-feature: fix code smells

* multiple-feature: fixes after review

---------

Co-authored-by: Oleksandr Hladchenko <[email protected]>
Co-authored-by: Oleksandr Hladchenko1 <[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.

1 participant