fix: adds labels for multiple selected artifacts

[antoine29]

Thank you for contributing to Harbor!

Comprehensive Summary of your change

When selecting multiple artifacts and adding a label for these, only the first selected artifact gets labeled. Current frontend code is adding the label only to the first selected artifact. This PR is fixing this by iterating the array and adding the label to each selected artifact.

Issue being fixed

Fixes #20681

Please indicate you've done the following:

• Well Written Title and Summary of the PR
• Label the PR as needed. "release-note/ignore-for-release, release-note/new-feature, release-note/update, release-note/enhancement, release-note/community, release-note/breaking-change, release-note/docs, release-note/infra, release-note/deprecation"
• Accepted the DCO. Commits without the DCO will delay acceptance.
• Made sure tests are passing and test coverage is added if needed.
• Considered the docs impact and opened a new docs issue or PR with docs changes if needed in website repository.

[reasonerjt]

@antoine29 Thanks for submitting the PR.
I don't have UI expertise so can't really comment on the code...
Could you explain how the errors are handled if the requests partially failed, i.e. some artifacts are labeled successfully but some failed?

[xuelichao]

The refresh() method will be called when the first callback is called, this is not expected.
We should call the refresh() method only when all requests are completed.
Maybe 'forkJoin' function can match our requirements.

this.selectedRow will be reset to empty array when the refresh() method is called.
this.selectedRow = []

if all the iteration is not done after the first request get completed, it will call the refresh() to reset the
this.selectedRow = []
Add/Remove labels will not be triggered for the items that not iterated before the selectedRow was reset to []

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.26%. Comparing base (c8c11b4) to head (7c4107b).
Report is 271 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main   #20873       +/-   ##
===========================================
+ Coverage   45.36%   66.26%   +20.89%     
===========================================
  Files         244     1046      +802     
  Lines       13333   114163   +100830     
  Branches     2719     2845      +126     
===========================================
+ Hits         6049    75653    +69604     
- Misses       6983    34386    +27403     
- Partials      301     4124     +3823     

Flag	Coverage Δ
unittests	66.26% <ø> (+20.89%)	⬆️

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

see 1286 files with indirect coverage changes

[antoine29]

@xuelichao thanks for the feedback. I'll look at the 'forkjoin' operator.

[github-actions]

This PR is being marked stale due to a period of inactivty. If this PR is still relevant, please comment or remove the stale label. Otherwise, this PR will close in 30 days.