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

MAUI-1205 Add facilities to penfaa project #4136

Merged
merged 1 commit into from
Sep 2, 2022

Conversation

IgorNadj
Copy link
Contributor

@IgorNadj IgorNadj commented Sep 1, 2022

Issue #: MAUI-1205

Changes:

  • Add facilities to penfaa project

Screenshots:

https://linear.app/bes/issue/MAUI-1205#comment-be5a6e22

Copy link
Contributor

@biaoli0 biaoli0 left a comment

Choose a reason for hiding this comment

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

LGTM.
@rohan-bes I think this migration could create duplicate records in Analytics and Ancestor tables due to the bug you are fixing (RN-635). Perhaps you can suggest the right process to merge in? It probably fine to merge into dev, just in case when release.

@biaoli0
Copy link
Contributor

biaoli0 commented Sep 2, 2022

@IgorNadj As discussed with @rohan-bes , it is safe to merge in.

@IgorNadj IgorNadj merged commit 81f1d0c into dev Sep 2, 2022
@IgorNadj IgorNadj deleted the maui-1205-add-facilities-penfaa branch September 2, 2022 06:54
@IgorNadj
Copy link
Contributor Author

IgorNadj commented Sep 5, 2022

Note: incorrectly merged with prefix MAUI-4136 in commit 81f1d0c, should have been MAUI-1205

IgorNadj added a commit that referenced this pull request Sep 6, 2022
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.

2 participants