[39] Addresses suggestions re iRODS file system setup UX

[ll4strw]

No description provided.

[ll4strw]

Please note that #44 wants to merge into branch rspace-os:irods which has also all other needed changes. If this is confusing, I can just make a different PR with all needed changes.

[ll4strw]

@rs-nicof commit 2023e55 implements the requested changes. Please note that I merged all the changes that I had regarding these enhancements into #44 to keep everything in one place. I hope this is not confusing.

[rs-nicof]

@rs-nicof commit 2023e55 implements the requested changes. Please note that I merged all the changes that I had regarding these enhancements into #44 to keep everything in one place. I hope this is not confusing.

 That's cool @ll4strw Thanks for addressing the comments.
 I would say better to close this PR, so we don't get confused 👍

[github-actions]

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

[mKowalski256]

This PR now has changes coming from #44, and should be good to review again. I'll start test AWS build so we can confirm it's all fine to merge.

[mKowalski256]

Re-tested #44 changes at https://rspace-os-irods-2.researchspace.com, they look good to me.

iRODS file system configuration panel before:

iRODS file system configuration panel after:

The PR is just js code changes to handle different label, and java code changes ensuring port 1247 for iRODS if no port is defined.
Unless anyone has any comments, I'll merge this PR before next release.

[mKowalski256]

Also @ll4strw please can you follow request of a "CLA Assistant Lite bot" from a few comments up, and add a Pull Request comment confirming you accept our CLA?

[ll4strw]

@mKowalski256 Thanks for pulling this PR in. I am running rspace's CLA through our copyright/patent office for approval first.

[mKowalski256]

I've re-checked the PR after latest merges, works fine for me. I see that on System screen there is new radio toggle between Native/PAM password type, looks good to me:

I also confirmed that previously-created iRODS configs work fine after change, and that screens for configuring Samba/SFTP behave the same. All looks good, so I'll merge now.