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.

Sign up for GitHub

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

Jump to bottom

hotfix/wrap configurations on slice store #19

Merged
merged 7 commits into from
Jan 1, 2025
Merged

hotfix/wrap configurations on slice store #19

merged 7 commits into from
Jan 1, 2025

Conversation

danifromecuador
Copy link
Owner

@danifromecuador danifromecuador commented Jan 1, 2025 •
edited
Loading

In this PR, I've:

  • Created a hotfix/wrap-configurations-on-slice-store branch
  • Nested language config options inside configs slice
  • Updated all the components that had references to language options
  • Fix typo errors on common tasks
  • Chante action name of set current language on redux actions

@danifromecuador
Nest language config options inside configurationOptionsSlice
8288ccf 
hotfix/wrap-configurations-on-slice-store
@danifromecuador
Replace reference of store.language for store.configs.language()
0139780 
hotfix/wrap-configurations-on-slice-store
@danifromecuador
Change language() for language:{} on option slice in store
7edf817 
hotfix/wrap-configurations-on-slice-store
@danifromecuador
Update all components that use the store value configs/language
3ca0f2f 
hotfix/wrap-configurations-on-slice-store
@danifromecuador
Create TODO code for commonTasks on configs slice
152efb3 
hotfix/wrap-configurations-on-slice-store
@danifromecuador
Fix typo error
061ef75 
hotfix/wrap-configurations-on-slice-store
@danifromecuador
Change action name of set current language on store
b9b0298 
hotfix/wrap-configurations-on-slice-store
@danifromecuador danifromecuador merged commit 9abf6f2 into develop Jan 1, 2025
3 checks passed
@danifromecuador danifromecuador deleted the hotfix/wrap-configurations-on-slice-store branch January 1, 2025 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@danifromecuador