Customizable PWA icon support

[gayanSandamal]

Description

Related Issue(s)

#1328

Notes

• Icons doesn't need to come from the FlowFuse assets as this supports icons from any origin
• Docs and built-in help are updated in a previous PR Added app icon field to ui base #1334
• Need to restart the node-red server to apply the new icon

Checklist

• I have read the contribution guidelines
• Suitable unit/system level tests have been added and they pass
• Documentation has been updated
  • Upgrade instructions
  • Configuration details
  • Concepts
• Changes flowforge.yml?
  • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
  • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production

Labels

• Includes a DB migration? -> add the area:migration label

[joepavitt]

This branch doesn't seem to contain the code in order to configure the icon?

[gayanSandamal]

This branch doesn't seem to contain the code in order to configure the icon?

This is my bad. I forgot to get changes from the main branch (#1144-configurable-pwa-icon) related with this issue. Just updated.

This is ready for review

[Steve-Mcl]

Looks good to me.

Question, shouldnt the name come from the base node Name field?

As it stands, all of my dashboards are called "Node-RED Dashboard 2.0" - not a great UX IMHO.

This can of course be a follow up task.

[gayanSandamal]

Looks good to me.

Question, shouldnt the name come from the base node Name field?

As it stands, all of my dashboards are called "Node-RED Dashboard 2.0" - not a great UX IMHO.

This can of course be a follow up task.

@Steve-Mcl As discussed a followup task has been raised #1375