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

feat: case floor plan pins editing #881

Merged
merged 6 commits into from
Sep 30, 2019

Conversation

nbiton
Copy link
Contributor

@nbiton nbiton commented Sep 20, 2019

Takes care of the pins editing portion of #855

image
image
image
image

@nbiton nbiton temporarily deployed to unee-t-pr-881 September 20, 2019 22:48 Inactive
@nbiton nbiton temporarily deployed to unee-t-pr-881 September 23, 2019 09:48 Inactive
@franck-boullier
Copy link
Member

feat: allow to add floor plan pins to a case after creation <-- Tested. This works as intended.

@nbiton nbiton temporarily deployed to unee-t-pr-881 September 24, 2019 03:55 Inactive
@nbiton nbiton temporarily deployed to unee-t-pr-881 September 24, 2019 14:04 Inactive
@nbiton
Copy link
Contributor Author

nbiton commented Sep 30, 2019

The remainder of #855 is implemented now. @franck-boullier please review

@franck-boullier
Copy link
Member

@nbiton it seems that the deployment did not work as expected. Can you confirm that I can test on Heroku now?

@nbiton
Copy link
Contributor Author

nbiton commented Sep 30, 2019

I just made it redeploy. You can try again in a few minutes

@franck-boullier
Copy link
Member

merge to test on the DEV/Staging

@franck-boullier franck-boullier merged commit 5e6a62d into unee-t:master Sep 30, 2019
This pull request was closed.
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