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

Use local storage to remember user choices #1459

Closed
wants to merge 2 commits into from
Closed

Use local storage to remember user choices #1459

wants to merge 2 commits into from

Conversation

4121nnn
Copy link

@4121nnn 4121nnn commented May 3, 2024

Caching implemented to these fields:
Project, Language, Spring boot, Packaging and Java version.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label May 3, 2024
@mhalbritter
Copy link
Contributor

mhalbritter commented May 3, 2024

Hey @4121nnn, i pushed a commit to this PR which polishes a bit. What do you think of the changes?

I moved all the localStorage storing into the event listeners instead of fiddling with the update function, and refactored code in some methods for better readability.

I also removed the storage of the selected Boot version. I don't think that this brings much value, as the versions change every month.

@mhalbritter mhalbritter changed the title Use local storage to remember user choices #1403 Use local storage to remember user choices May 3, 2024
@mhalbritter mhalbritter added type: enhancement status: waiting-for-feedback We need additional information before we can continue and removed status: waiting-for-triage An issue we've not yet triaged labels May 3, 2024
@4121nnn
Copy link
Author

4121nnn commented May 3, 2024

Thank you for improvements @mhalbritter . Your changes looks much better than my code

@spring-projects-issues spring-projects-issues added status: feedback-provided Feedback has been provided and removed status: waiting-for-feedback We need additional information before we can continue labels May 3, 2024
@mhalbritter mhalbritter requested a review from oodamien May 3, 2024 10:09
@oodamien
Copy link
Contributor

oodamien commented May 6, 2024

Closing in favour of #1462

@mhalbritter
Copy link
Contributor

Hey @4121nnn, thanks for the PR. I had @oodamien looked at the PR, who has more understanding of React than I do. The local storage setting should be done in the reducer / context. He opened #1462 for that. Thanks anyway!

@4121nnn
Copy link
Author

4121nnn commented May 6, 2024

No problem @mhalbritter, actually it is my first pull request ever. I will keep improving my skills. But thanks anyway for letting me participate in this project , I really appreciate that

@mhalbritter
Copy link
Contributor

mhalbritter commented May 6, 2024

Keep 'em coming! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants