Closed arc6789 closed 10 months ago
@zoobot The images got included in this PR. I am assuming we don't want those checked in? Because there are a ALOT...
@zoobot The images got included in this PR. I am assuming we don't want those checked in? Because there are a ALOT...
Yes please remove.
Also, I think you will need to update the snapshot so the tests work. Let me know if you need help with that. I think we have docs on it. https://github.com/waterthetrees/waterthetrees/wiki/Testing
I don't see docs on how to update but the flag to update the snapshot is -u
@zoobot The images got included in this PR. I am assuming we don't want those checked in? Because there are a ALOT...
Yes please remove.
Also, I think you will need to update the snapshot so the tests work. Let me know if you need help with that. I think we have docs on it. https://github.com/waterthetrees/waterthetrees/wiki/Testing
I don't see docs on how to update but the flag to update the snapshot is -u
I have updated the snapshot as well as the text!
@arc6789 I am not seeing this on the dev server. I merged and pulled latest code, ran the build. Should I see it? https://dev.waterthetrees.com
I can see the new sidepanel given the link above. Maybe you need to clear your cache?
Description:
Moving the maintenance dialog form into a sidepanel within the tree details section. You can access this form when clicking on the maintenance button. Details including mockups and feature requirements are included here https://github.com/waterthetrees/wtt_front/issues/564
New maintenance side panel:
Success modal after form has been submitted:
Maintenance information on the tree details section: