pfitzpaddy / ngm-reportDesk

The workdesk for ReportHub
1 stars 6 forks source link

Update Target Location Interface from xeditable to Forms #310

Closed drfaustusfade closed 4 years ago

drfaustusfade commented 4 years ago

already in works @fakhrihawari ApplicationFrameHost_2020-01-20_13-57-18

to be updated for AF

fakhrihawari commented 4 years ago

for old materializecss https://github.com/fakhrihawari/ngm-reportHub/tree/change-location-in-detail for materializecss 1x https://github.com/fakhrihawari/ngm-reportHub/tree/check-error-for-materialize-100

fakhrihawari commented 4 years ago

How it will look like:

Screen Shot 2020-01-21 at 16 38 12 Screen Shot 2020-01-21 at 16 39 55
pfitzpaddy commented 4 years ago

We will need to test Target Locations for;

Im guessing that the current implementation for Afghanistan is based on the implementation for Cox's Bazar?

fakhrihawari commented 4 years ago

Yes

pfitzpaddy commented 4 years ago

Nice work. Can you tick off the countries that you deem to be working. Meanwhile I will commit the upgrade to Materialize 1.x today

drfaustusfade commented 4 years ago

We were planning to update for AF first, and then others if they test it ok, but they mostly the same, NG has from list selection, which was included, also higher admins, implementation fields based on old ones

pfitzpaddy commented 4 years ago

Oh yes, good point. We can commit Afghanistan first with others using old approach until they are tested / moved over....

pfitzpaddy commented 4 years ago

If all good for AF, you can make a pull request for the 1.x