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

MINT-1788 Remove ui actions from core-dependencies #791

Merged

Conversation

olegdeezus
Copy link
Contributor

@olegdeezus olegdeezus commented Sep 10, 2018

redux/modals is a pure state part. It is needed by some core actions
and can't be removed from core totally.
Modals reducer can be replaced in different environment.

@olegdeezus olegdeezus changed the title MINT-1788 Remove ui actions from core-dependencies [WIP] MINT-1788 Remove ui actions from core-dependencies Sep 10, 2018
@coveralls
Copy link

coveralls commented Sep 10, 2018

Coverage Status

Coverage decreased (-0.5%) to 0.942% when pulling 1d86ba4 on olegdizus:feature/mint-1788-remove-ui-actions into 2bb2d4d on ChronoBank:develop.

redux/modals is a pure state part. It is needed by some core actions
and can't be removed from core totally.
Modals reducer can be replaced in different environment.
@olegdeezus olegdeezus force-pushed the feature/mint-1788-remove-ui-actions branch from 662c397 to 3f5662f Compare September 10, 2018 10:50
@olegdeezus olegdeezus changed the title [WIP] MINT-1788 Remove ui actions from core-dependencies MINT-1788 Remove ui actions from core-dependencies Sep 10, 2018
Copy link
Contributor

@ozalexo ozalexo left a comment

Choose a reason for hiding this comment

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

Oleg, why methods modalsOpen and other web-related methods were moved back into @core package?

ozalexo
ozalexo previously approved these changes Sep 10, 2018
Copy link
Contributor

@ozalexo ozalexo left a comment

Choose a reason for hiding this comment

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

LGTM

e-maximus
e-maximus previously approved these changes Sep 10, 2018
@olegdeezus olegdeezus dismissed stale reviews from e-maximus and ozalexo via 1d86ba4 September 10, 2018 13:41
@olegdeezus olegdeezus force-pushed the feature/mint-1788-remove-ui-actions branch from de68ee7 to 1d86ba4 Compare September 10, 2018 13:41
Copy link
Contributor

@ozalexo ozalexo left a comment

Choose a reason for hiding this comment

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

LGTM
Discussed all details. It is OK for now.

@ozalexo ozalexo merged commit e0d1b82 into Chrono-Tech:develop Sep 10, 2018
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.

None yet

4 participants