Skip to content

Improve Usablility for changing User Access Hosts - #5676

Merged
greenbonebot merged 3 commits into
mainfrom
user-access-hosts
Oct 1, 2026
Merged

greenbonebot merged 3 commits into
mainfrom
user-access-hosts

Conversation

@bjoernricks

@bjoernricks bjoernricks commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

What

Improve Usablility for changing User Access Hosts by using a tagging like text input field.

Also update all user create and save variables to camelCase.

Why

The access hosts contain a CSV value which is actually a list of values. Therefore represent the values as an array of strings.

References

https://jira.greenbone.net/browse/GEA-1973

Checklist

  • I have added tests for the changes
  • I have used the following LLMs/AI tools in this pull request:

@bjoernricks
bjoernricks requested a review from a team as a code owner October 1, 2026 08:26
@github-actions

github-actions Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 677db13.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@greenbonebot
greenbonebot enabled auto-merge (rebase) October 1, 2026 08:26
@bjoernricks
bjoernricks marked this pull request as draft October 1, 2026 09:08
auto-merge was automatically disabled October 1, 2026 09:08

Pull request was converted to draft

@bjoernricks
bjoernricks marked this pull request as ready for review October 1, 2026 09:12
@greenbonebot
greenbonebot enabled auto-merge (rebase) October 1, 2026 09:12
@codecov

codecov Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.43%. Comparing base (38bbd98) to head (677db13).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5676      +/-   ##
==========================================
+ Coverage   87.36%   87.43%   +0.06%     
==========================================
  Files        1304     1304              
  Lines       34292    34289       -3     
  Branches    10287    10284       -3     
==========================================
+ Hits        29959    29979      +20     
+ Misses       3919     3897      -22     
+ Partials      414      413       -1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Allow for better representation of CSV values in the UI. This improves
the usability of the UserDialog component. Actually the access hosts are
a list of string values. With this change everywhere in GSA the access
hosts are represented as an array and are only converted to CSV when
using the backend API.
Ensure saving and creating users via the User Dialog works as expected.
@greenbonebot
greenbonebot merged commit 92d7b11 into main Oct 1, 2026
23 checks passed
@greenbonebot
greenbonebot deleted the user-access-hosts branch October 1, 2026 10:33
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.

3 participants