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

Convert dropdown_menu state to Typescript #25585

Merged
merged 1 commit into from
Sep 22, 2023

Conversation

renchap
Copy link
Member

@renchap renchap commented Jun 26, 2023

This builds over my #25584 PR.

It shows how to convert from the legacy Immutable state to the new Redux Toolkit + Immer state, which work with plain objects.

It features:

  • typed actions
  • typed state, without Immutable
  • actions & reducer using RTK toolkit helpers
  • typing mapStateToProps in JS files

@renchap
Copy link
Member Author

renchap commented Jun 26, 2023

Ping @takayamaki if you have some suggestions or comments on the way I did it!

@renchap renchap force-pushed the typescript-dropdownmenu-state branch from 95e4594 to 7ddcc8e Compare July 4, 2023 09:44
Copy link
Contributor

@takayamaki takayamaki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall LGTM 👍

@renchap renchap marked this pull request as ready for review July 11, 2023 10:44
@renchap renchap requested a review from a team July 11, 2023 10:44
@renchap renchap force-pushed the typescript-dropdownmenu-state branch from 7ddcc8e to 0023cd4 Compare August 22, 2023 17:23
Using the proper Redux Toolkit helpers
@renchap renchap force-pushed the typescript-dropdownmenu-state branch from 0023cd4 to 2ed555d Compare September 17, 2023 19:51
@ClearlyClaire ClearlyClaire merged commit 4b7bc1f into mastodon:main Sep 22, 2023
audiodude pushed a commit to audiodude/mastodon that referenced this pull request Oct 23, 2023
000hen pushed a commit to thenapnetwork/nap-mastodon that referenced this pull request Dec 6, 2023
vmstan pushed a commit to vmstan/mastodon that referenced this pull request Jan 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants