Skip to content
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

[BUG] <title>modified config in the ui,resulting in an error in the configuration file #1403

Closed
5 tasks done
snbzy opened this issue Nov 30, 2023 · 5 comments
Closed
5 tasks done
Assignees
Labels
🐛 Bug [ISSUE] Ticket describing something that isn't working 🕸️ Duplicate [ISSUE] This issue or pull request already exists

Comments

@snbzy
Copy link

snbzy commented Nov 30, 2023

Environment

Self-Hosted (Docker)

System

No response

Version

No response

Describe the problem

When I edit the configuration file in the UI and click Save to Profile, the configuration file will be modified to the wrong format, resulting in the inability to rebuild the application
image

Additional info

No response

Please tick the boxes

@snbzy snbzy added the 🐛 Bug [ISSUE] Ticket describing something that isn't working label Nov 30, 2023
@liss-bot
Copy link
Collaborator

If you're enjoying Dashy, consider dropping us a ⭐
🤖 I'm a bot, and this message was automated

@snbzy
Copy link
Author

snbzy commented Dec 6, 2023

I've fixed this, please see the image below.You only need to serialize and re-deserialize the configured objects to ensure that the configuration file is not garbled
image

@liss-bot liss-bot added the 👤 Awaiting Maintainer Response [ISSUE] Response from repo author is pending label Dec 6, 2023
@CrazyWolf13
Copy link
Collaborator

CrazyWolf13 commented Feb 3, 2024

Duplicate of #1361, please close this

@liss-bot liss-bot removed the 👤 Awaiting Maintainer Response [ISSUE] Response from repo author is pending label Feb 4, 2024
@CrazyWolf13 CrazyWolf13 added the 🕸️ Duplicate [ISSUE] This issue or pull request already exists label Feb 22, 2024
@CrazyWolf13
Copy link
Collaborator

Thanks for providing your screenshot and a possibly fix.
I'll look into this and if it can confirm, i'll push a fix.

Will keep you updated!

@CrazyWolf13
Copy link
Collaborator

Thanks, fix is pushed and approved: #1470

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug [ISSUE] Ticket describing something that isn't working 🕸️ Duplicate [ISSUE] This issue or pull request already exists
Projects
Status: Done
Development

No branches or pull requests

4 participants