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

Maintenance/reducer restructure #182

Merged
merged 7 commits into from
Aug 24, 2022
Merged

Conversation

langdal
Copy link
Member

@langdal langdal commented Jul 8, 2022

This PR will restructure the file layout of the project.

  • Component specific reducers are moved into the respective component folders.
  • Global and experiment wide reducers are now collocated with the corresponding context providers
  • Module aliases are introduced so that imports can be rewritten like this import {ExperimentProvider} from '../../context/experiment' -> import {ExperimentProvider} from '@/context/experiment
    This will make imports stable when moving files around during refactorings

@langdal langdal self-assigned this Jul 8, 2022
@langdal langdal force-pushed the maintenance/reducer-restructure branch from 00aabd4 to 21c21ad Compare July 8, 2022 11:55
@langdal langdal marked this pull request as ready for review July 8, 2022 12:05
@langdal langdal force-pushed the maintenance/reducer-restructure branch from 21c21ad to 930fbfe Compare July 9, 2022 09:03
@langdal langdal force-pushed the maintenance/reducer-restructure branch from cb7e270 to afbfb54 Compare August 22, 2022 12:39
@langdal langdal requested a review from j-or August 22, 2022 21:56
@langdal langdal merged commit 40a3a85 into main Aug 24, 2022
@langdal langdal deleted the maintenance/reducer-restructure branch August 24, 2022 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant