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

Local properties storage do no reset #6279

Closed
DamnClin opened this issue May 16, 2023 · 6 comments · Fixed by #6314
Closed

Local properties storage do no reset #6279

DamnClin opened this issue May 16, 2023 · 6 comments · Fixed by #6314
Assignees
Labels

Comments

@DamnClin
Copy link
Collaborator

The local properties storage do not reset when changing folder wich is really annoying since, when you start a new application, you still have properties from the previous one. We need to keep properties only for the same application

@DamnClin DamnClin added area: bug 🐛 Something isn't working generator: internal labels May 16, 2023
@DamnClin
Copy link
Collaborator Author

cc @renanfranca can you take it?

@renanfranca
Copy link
Collaborator

@DamnClin sure! I'll do as soon as possible 👍

@DamnClin
Copy link
Collaborator Author

No rush, this is annoying but usable (just a doubt that you'll see more properties than needed)

@renanfranca
Copy link
Collaborator

Hello @DamnClin,

today when click on Module Patch or Landscape menu the folder path is refreshed with a random directory. if the local storage got clear when the folder path change, it will be clear every time.

Can I store the folder path at the local storage after it got generated? What do you think about it?

@DamnClin
Copy link
Collaborator Author

You need to have a key depending on the path because we can change path whenever we want :)

@renanfranca
Copy link
Collaborator

You need to have a key depending on the path because we can change path whenever we want :)

Thank you, I will store the folder path with a specific key each time it gets changed and clear the properties from storage before loading the module's history ✌️.

renanfranca added a commit to renanfranca/jhipster-lite that referenced this issue May 18, 2023
renanfranca added a commit to renanfranca/jhipster-lite that referenced this issue May 29, 2023
renanfranca added a commit to renanfranca/jhipster-lite that referenced this issue May 29, 2023
renanfranca added a commit to renanfranca/jhipster-lite that referenced this issue May 30, 2023
renanfranca added a commit to renanfranca/jhipster-lite that referenced this issue May 31, 2023
DamnClin pushed a commit that referenced this issue Jun 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants
@renanfranca @DamnClin and others