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

refactor: cleanup state #227

Merged
merged 4 commits into from
Jun 15, 2020
Merged

refactor: cleanup state #227

merged 4 commits into from
Jun 15, 2020

Conversation

dhhyi
Copy link
Collaborator

@dhhyi dhhyi commented May 3, 2020

PR Type

[x] Refactoring (no functional changes, no API changes)

What Is the Current Behavior?

Core State has many substates that could be grouped together.

What Is the New Behavior?

grouping

Does this PR Introduce a Breaking Change?

[x] Yes
[ ] No

@dhhyi dhhyi added the refactoring Refactoring of current code label May 3, 2020
@dhhyi dhhyi self-assigned this May 3, 2020
@dhhyi dhhyi force-pushed the refactor/cleanup-core-state branch 2 times, most recently from 0c80416 to 790a062 Compare May 3, 2020 13:44
@dhhyi dhhyi changed the title refactor: cleanup core state refactor: cleanup state May 3, 2020
@dhhyi dhhyi force-pushed the refactor/ngrx-testing branch 2 times, most recently from 2c5ae73 to 5c33a95 Compare May 3, 2020 21:48
@dhhyi dhhyi force-pushed the refactor/cleanup-core-state branch from d190efc to 012ec05 Compare May 3, 2020 23:27
@dhhyi dhhyi force-pushed the refactor/ngrx-testing branch from 5c33a95 to 96dbb6a Compare May 4, 2020 09:06
@dhhyi dhhyi force-pushed the refactor/cleanup-core-state branch from 012ec05 to b6421bf Compare May 4, 2020 10:12
@dhhyi dhhyi added this to the 0.21 milestone May 7, 2020
@dhhyi dhhyi force-pushed the refactor/ngrx-testing branch from 96dbb6a to 101a810 Compare May 12, 2020 21:23
@dhhyi dhhyi force-pushed the refactor/cleanup-core-state branch from 94f7c05 to fc6dc45 Compare May 14, 2020 12:05
@dhhyi dhhyi force-pushed the refactor/ngrx-testing branch from 101a810 to 8ada488 Compare May 14, 2020 12:52
@dhhyi dhhyi force-pushed the refactor/cleanup-core-state branch 3 times, most recently from ab09b12 to 40bcebf Compare May 14, 2020 14:57
@dhhyi dhhyi force-pushed the refactor/ngrx-testing branch from 8ada488 to d934f93 Compare May 25, 2020 20:44
@dhhyi dhhyi force-pushed the refactor/cleanup-core-state branch from 40bcebf to fdeb386 Compare May 26, 2020 11:30
@dhhyi dhhyi force-pushed the refactor/ngrx-testing branch from d934f93 to ecea3db Compare May 28, 2020 20:55
@dhhyi dhhyi force-pushed the refactor/cleanup-core-state branch from fdeb386 to de0fd9d Compare May 28, 2020 21:06
@dhhyi dhhyi force-pushed the refactor/ngrx-testing branch 2 times, most recently from c8e3626 to e98f1c1 Compare May 30, 2020 23:29
@dhhyi dhhyi force-pushed the refactor/cleanup-core-state branch 6 times, most recently from 60bff66 to 4166f1f Compare June 2, 2020 11:20
@dhhyi dhhyi force-pushed the refactor/cleanup-core-state branch from 7bb5b39 to d391ff4 Compare June 7, 2020 22:29
@dhhyi dhhyi changed the base branch from refactor/ngrx-testing to develop June 8, 2020 19:24
@dhhyi dhhyi force-pushed the refactor/cleanup-core-state branch from d391ff4 to 25ace4f Compare June 8, 2020 19:27
@dhhyi dhhyi requested review from SGrueber and MWallnisch June 8, 2020 19:27
MWallnisch
MWallnisch previously approved these changes Jun 9, 2020
@dhhyi dhhyi force-pushed the refactor/cleanup-core-state branch from ee6052a to ef32adf Compare June 9, 2020 18:25
@dhhyi dhhyi force-pushed the refactor/cleanup-core-state branch from ef32adf to b6557b3 Compare June 11, 2020 14:34
@dhhyi dhhyi merged commit b2ec14a into develop Jun 15, 2020
@dhhyi dhhyi deleted the refactor/cleanup-core-state branch June 15, 2020 07:11
dhhyi added a commit that referenced this pull request Feb 26, 2021
- needed for access to locale setting of ICM in core configuration state
- relates to #227
dhhyi added a commit that referenced this pull request Feb 26, 2021
- needed for access to locale setting of ICM in core configuration state
- relates to #227
dhhyi added a commit that referenced this pull request Feb 26, 2021
- needed for access to locale setting of ICM in core configuration state
- relates to #227
dhhyi added a commit that referenced this pull request Mar 3, 2021
- needed for access to locale setting of ICM in core configuration state
- relates to #227
dhhyi added a commit that referenced this pull request Apr 18, 2021
- needed for access to locale setting of ICM in core configuration state
- relates to #227
dhhyi added a commit that referenced this pull request Apr 18, 2021
- needed for access to locale setting of ICM in core configuration state
- relates to #227
dhhyi added a commit that referenced this pull request Apr 18, 2021
- needed for access to locale setting of ICM in core configuration state
- relates to #227
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Refactoring of current code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants