If nomad_helper_url is not available user should dismissable popover

[piotrzarzycki21]

When user clicks on Nomad link and nomad_helper_url is not available user should get dismissable popover with message:

• Popover should appear only when user clicks for the first time in Nomad link
• State of that popover will be saved in cookies. If user clean cookies, popover appear again after click on Nomad link.

[JoelProminic]

We also want to compare the page for nomad_helper_url to the expected nomadhelper.html and show a similar warning if they don't match exactly. Genesis doesn't have access to install nomad_helper.html, so we want to have SuperHumanPortal verify that the nomad_helper.html installed by the administrator matches the copy under resources/.

If we are reading nomad_helper_url, then we need to safely handle the case where the URL doesn't resolve properly (because it was not properly configured). I would expect the administrator to test this, but until it is fixed, Super.Human.Portal should use the default logic instead.

A mismatching nomad_helper.html is not necessarily a blocker, so I think we should attempt to use the helper anyway in this case. If this fails, it should be handled by the callback logic.

[piotrzarzycki21]

I have added warning window. With newest build it should be available for tests.

[JoelProminic]

@JustinProminic proposed that we add detailed setup instructions for nomadhelper.html in the popup. For example:

This link is currently configured to open in a separate HCL Nomad Web window. For the best user experience, the administrator needs to setup nomadhelper.nsf like this:

• Copy %data_directory%/domino/html/Super.Human.Portal/js-release/resources/nomadhelper.html to /opt/hcl/domino/notes/latest/linux/nomad-files [Copy to Cllpboard] on your Nomad server
• Update nomad_helper_url to %nomad_base_url%/nomadhelper.html [Copy to Clipboard] in SuperHumanPortal.nsf [Open in Notes] [Open in Nomad]

This will require that I update ConfigRead with at least these properties:

• data_directory - the Domino data directory
• nomad_base_url - user-configured base URL for Nomad
• Notes and Nomad URLs for SuperHumanPortal.nsf/Configuration?OpenView
• server_name? This may be generally useful for building links.

[JoelProminic]

I updated the ConfigRead agent with the required values:

Property	Notes
domino_data_directory	Currently hard-coded to /local/notesdata. I have some code to read this dynamically, but it feels like overkill.
domino_server_name	Abbreviated format, so it should be ready for display
nomad_base_url	The base URL for Nomad. nomad_helper_url (if populated) should be a superstring of this.
configuration_link	A link to the Configuration view in SuperHumanPortal.nsf

Example:

{
  "errorMessage": "",
  "state": "authenticated",
  "common_name": "Anonymous",
  "config": {
    "ui_title": "Super.Human.Portal",
    "ui_documentation_editable": false,
    "nomad_base_url": "https://nomadweb.nomad-1010.nomadtest.com/nomad",
    "domino_data_directory": "/local/notesdata",
    "configuration_link": {
      "server": "nomad-1010.nomadtest.com/NOMADTEST",
      "database": "SuperHumanPortal.nsf",
      "view": "Configuration",
      "defaultAction": "nomad",
      "name": "SuperHumanPortal Configuration",
      "description": "This is an example link description for Super.Human.Portal.",
      "nomadURL": "https://nomadweb.nomad-1010.nomadtest.com/nomad/#/notes://nomad-1010.nomadtest.com/SuperHumanPortal.nsf/Configuration?OpenView",
      "type": "database",
      "url": "notes://nomad-1010.nomadtest.com/SuperHumanPortal.nsf/Configuration?OpenView"
    },
    "ui_documentation_show_unid": false,
    "domino_server_name": "nomad-1010.nomadtest.com/NOMADTEST",
    "nomad_helper_url": "https://nomadweb.nomad-1010.nomadtest.com/nomad/nomadhelper.html"
  },
  "username": "Anonymous",
  "status": "authenticated"
}

[piotrzarzycki21]

@JoelProminic I have merged your changes to main. Let's work on main towards release. I have pushed changes which showing custom window. It requires more work on formatting text - if I found time later today I will try to adjust this. Let me know if this is enough good.

[piotrzarzycki21]

Question: I'm not sure what actually buttons copy should copy. Should I copy full text which is near button or less ?

[JoelProminic]

Yeah, I was expecting the [Copy to Clipboard] buttons to copy the corresponding path. I see I missed a button in the first line, though.

If we find this button doesn't fit well, we could remove them, but I'd like to see the paths formatted to stand out either way (for example, as monospaced text, or even as a separate block).

[piotrzarzycki21]

Yeah, I was expecting the [Copy to Clipboard] buttons to copy the corresponding path. I see I missed a button in the first line, though.

This is aligned with other places where we have copy to clipboard like that.

[piotrzarzycki21]

Yeah, I was expecting the [Copy to Clipboard] buttons to copy the corresponding path. I see I missed a button in the first line, though.

This is aligned with other places where we have copy to clipboard like that.

[piotrzarzycki21]

@JoelProminic I have wrapped paths to tag and changed color - if it's too drastic let me know. :)

[piotrzarzycki21]

I have spent some time and improve look of that popup in dark mode. In light mode it changes a bit as well. Let me know if new build is acceptable.

[JoelProminic]

The updated prompt looks like this:

Notes:

• For the first line, I was expecting to see the copy-to-clipboard button for both paths, so that they could be copied separately. Currently the full line is copied.
• The "Open in Notes" button should be "Open in Client" to match the format for bookmarks.
• This looks like a modal prompt, but it does not actually block me from doing some actions in the background (Copy, Refresh, select a database, etc.). This is a bit strange, but not a problem if is not causing other bugs. The close button works fine.
• Otherwise, this is looking fine in both light and dark mode.
• The prompt is only showing up once, as expected. This is controlled by the SuperHumanPortalNomadHelperUrlLinkWarning cookie, which can be cleared to retest

[piotrzarzycki21]

• For the first line, I was expecting to see the copy-to-clipboard button for both paths, so that they could be copied separately. Currently the full line is copied.
• The "Open in Notes" button should be "Open in Client" to match the format for bookmarks.

@JoelProminic I have made an adjustment for these two items. It's available in the next build.

[JoelProminic]

The updates look good. The remaining issue for comparing nomadhelper.html has been moved to #57.