Skip to content

Can't save changes in layout editor on FFC (Error 401) #1598

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Steve-Mcl opened this issue Jan 24, 2025 · 1 comment
Open

Can't save changes in layout editor on FFC (Error 401) #1598

Steve-Mcl opened this issue Jan 24, 2025 · 1 comment
Labels
bug Something isn't working needs-triage Needs looking at to decide what to do

Comments

@Steve-Mcl
Copy link
Contributor

Current Behavior

Image

Expected Behavior

should save changes

Steps To Reproduce

open a FFC instance, edit a layout, hit save.

Environment

  • Dashboard version: 1.22.0
  • Node-RED version: 4.0.8
  • Node.js version:
  • npm version:
  • Platform/OS: win 11
  • Browser: Chrome

Have you provided an initial effort estimate for this issue?

I have provided an initial effort estimate

@Steve-Mcl Steve-Mcl added bug Something isn't working needs-triage Needs looking at to decide what to do labels Jan 24, 2025
@Steve-Mcl
Copy link
Contributor Author

We are seeing more and more users (both FFC and Discourse) hitting issues around dashboard editing.

During development I first implemented this differently and feel we may wish to revisit this topic...

When I took over the great foundation Joe kicked off, I hit an issue with API tokens (similar to the one Paul had recently and TBH, there are probably a bunch more edge cases out there) so I changed the workflow to emit the edits to Node-RED WITHOUT deploying them. The user had switch back to Node-RED and press "Deploy" to apply the changes. That eliminated the majority of the friction around merging but would have introduced different UX friction. To get past that poor UX, in my head, the workfow needed to be something like:

User choses to edit a page - a shading overlay dialog covers Node RED and displays "Dashboard Editing Page xxx"
The page to edit is popped out to a separate window (as it is now)
User goes about edits and eventually saves or exits, which returned the user to Node-RED where they would see the shaded dialog once more
The shading dialog would now display the edits that are held in memory (e.g. "Page xxx has undeployed edits") and at this point the user choses to Deploy or Discard the edits.
This meant there would be ZERO back door deploys via API & zero merge issues - everything is handled by the normal Node-RED deploy mechanism.

Ref: #1484 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs-triage Needs looking at to decide what to do
Projects
Status: Backlog
Development

No branches or pull requests

1 participant