unee-t / frontend

Meteor front end
https://case.dev.unee-t.com/
GNU Affero General Public License v3.0
9 stars 17 forks source link

Case pin on floor plan #874

Closed nbiton closed 4 years ago

nbiton commented 4 years ago

Partially addresses #855

This PR includes the creation of pins on an existing floor plan in the case creation view, and display of the floor plan with pins in the case messages view.

Functionality not included in this PR which will be implemented separately:

franck-boullier commented 4 years ago

Tests Done:

Test that went OK:

Tests that failed:

Unable to test:

franck-boullier commented 4 years ago

The test done:

Test that went OK:

Tests that failed:

Unable to test:

nbiton commented 4 years ago

Move the pin on a floor plan for a case

@franck-boullier Why would you need to move a pin, if you can just remove the previous and add a new one?

After I zoom in on the floor plan it is NOT possible to move the image around. Some part of the floor plan are NOT visible when the floor plan is zoomed in

I don't understand what this means

I can see zoom in on the floor plan in the case the conversation

I'm not sure I understand exactly what this means, but the floor plan on the messages view is currently designed to be static. The user won't be able to zoom in on the floor plan in the message view

nbiton commented 4 years ago

This PR should be ready to merge now

franck-boullier commented 4 years ago

Basic functionalities tested and OK:

Logged in as Nath

The following functionalities will come in a separate PR: