drfaustusfade / ngm-reportDesk

The workdesk for ReportHub
0 stars 1 forks source link

Organization Page #25

Closed fakhrihawari closed 3 years ago

fakhrihawari commented 3 years ago

Switch Button

https://user-images.githubusercontent.com/12655589/103966023-97927e00-5191-11eb-9606-f576a0933f6d.mp4

Mock Up 192 168 33 16_desk_ (33)

drfaustusfade commented 3 years ago

Alright, almost there,

Organization Profile

fakhrihawari commented 3 years ago
Screen Shot 2021-01-11 at 16 42 50 Screen Shot 2021-01-11 at 16 42 43
fakhrihawari commented 3 years ago

192 168 33 16_desk_ (35)

fakhrihawari commented 3 years ago

is it like that ?

drfaustusfade commented 3 years ago
fakhrihawari commented 3 years ago
Screen Shot 2021-01-12 at 10 04 01 Screen Shot 2021-01-12 at 10 03 51

Is it okay ?

fakhrihawari commented 3 years ago

192 168 33 16_desk_ (36)

drfaustusfade commented 3 years ago

alright, nice, nearly there

fakhrihawari commented 3 years ago

point 1 what do you mean centre? vertically? point 2 can you give me an example

fakhrihawari commented 3 years ago

point 3

Screen Shot 2021-01-12 at 13 48 39
drfaustusfade commented 3 years ago

point 1 vertically point 2 on card view having just Organizations Contacts column with card details of all reported organizations point 3 how would it look if the collection would be on a card without dashboard label with small margins

fakhrihawari commented 3 years ago

point 1 , 2

https://user-images.githubusercontent.com/12655589/104284891-5da0df00-54e5-11eb-9d2f-83ece4409bc2.mp4

remove duplicate organization ? or keep them, coz in the organization for each cluster, the reporter is different

fakhrihawari commented 3 years ago

point3

Screen Shot 2021-01-12 at 14 57 26
drfaustusfade commented 3 years ago

point 1 , 2

2demo.mp4 remove duplicate organization ? or keep them, coz in the organization for each cluster, the reporter is different

very good point, more generally the there could be multiple reporters per cluster, it that case putting a label or title on each cluster section might work

drfaustusfade commented 3 years ago

point3

Screen Shot 2021-01-12 at 14 57 26

alright, maybe also putting some heading with Pages

fakhrihawari commented 3 years ago
Screen Shot 2021-01-12 at 15 20 14
drfaustusfade commented 3 years ago

cool, maybe just one of those or putting cluster under organization name, also, i just noticed if we could please put user full name before email

fakhrihawari commented 3 years ago
Screen Shot 2021-01-12 at 15 52 45
drfaustusfade commented 3 years ago

alright, looks good

fakhrihawari commented 3 years ago
Screen Shot 2021-01-12 at 16 20 52
drfaustusfade commented 3 years ago

ok, i think good to go like that

fakhrihawari commented 3 years ago

Link for PR https://github.com/drfaustusfade/ngm-reportEngine/pull/15 https://github.com/drfaustusfade/ngm-reportHub/pull/54

drfaustusfade commented 3 years ago
fakhrihawari commented 3 years ago

Point 1 what do you mean?

fakhrihawari commented 3 years ago

Updated

fakhrihawari commented 3 years ago
Screen Shot 2021-01-13 at 16 52 50
drfaustusfade commented 3 years ago
drfaustusfade commented 3 years ago
drfaustusfade commented 3 years ago

separate task/pull request

fakhrihawari commented 3 years ago

separate task/pull request

  • [ ] country organization list ( to quickly check organizations present in the country, main users Cluster and Country Admins ) Title : AF | Organization download: list of organizations table: Organization | Organization (Abrv) | Type | Joined | Last Reported | Clusters | Focal Point Name | Email | Phone | (Link to organization page) filter: by Type | Cluster the cluster based on organization-defined organization, ( that will be used for Cluster filter to get Last Reported ) roles: excluding User, Org ( that is closed access for now )

Is just for AF or all country too? if for all country need to put country filter too

drfaustusfade commented 3 years ago

the main use for the page is per country, not required but the country filter can be put also

separate task/pull request

  • [ ] country organization list ( to quickly check organizations present in the country, main users Cluster and Country Admins ) Title : AF | Organization download: list of organizations table: Organization | Organization (Abrv) | Type | Joined | Last Reported | Clusters | Focal Point Name | Email | Phone | (Link to organization page) filter: by Type | Cluster the cluster based on organization-defined organization, ( that will be used for Cluster filter to get Last Reported ) roles: excluding User, Org ( that is closed access for now )

Is just for AF or all country too? if for all country need to put country filter too

not required, the main page use is per country but can be put also

fakhrihawari commented 3 years ago

the main use for the page is per country, not required but the country filter can be put also

separate task/pull request

  • [ ] country organization list ( to quickly check organizations present in the country, main users Cluster and Country Admins ) Title : AF | Organization download: list of organizations table: Organization | Organization (Abrv) | Type | Joined | Last Reported | Clusters | Focal Point Name | Email | Phone | (Link to organization page) filter: by Type | Cluster the cluster based on organization-defined organization, ( that will be used for Cluster filter to get Last Reported ) roles: excluding User, Org ( that is closed access for now )

Is just for AF or all country too? if for all country need to put country filter too

not required, the main page use is per country but can be put also

the clusters attribute in organization exist? or its cluster from last report?

fakhrihawari commented 3 years ago
Screen Shot 2021-01-14 at 14 35 24
drfaustusfade commented 3 years ago

the main use for the page is per country, not required but the country filter can be put also

separate task/pull request

  • [ ] country organization list ( to quickly check organizations present in the country, main users Cluster and Country Admins ) Title : AF | Organization download: list of organizations table: Organization | Organization (Abrv) | Type | Joined | Last Reported | Clusters | Focal Point Name | Email | Phone | (Link to organization page) filter: by Type | Cluster the cluster based on organization-defined organization, ( that will be used for Cluster filter to get Last Reported ) roles: excluding User, Org ( that is closed access for now )

Is just for AF or all country too? if for all country need to put country filter too

not required, the main page use is per country but can be put also

the clusters attribute in the organization exist? or its cluster from the last report?

clusters from organization array attribute ( so that organizations define which clusters currently they are affiliated ) if all clusters ( last reported beneficiaries record from that organization with their clusters ( if no yet clusters defined then any organization record ) if filtered cluster ( last reported cluster beneficiaries record from that organization )

fakhrihawari commented 3 years ago

Update organization lists

Screen Shot 2021-01-15 at 16 06 32 Screen Shot 2021-01-15 at 16 06 21
drfaustusfade commented 3 years ago

alright, also, in the menu let's put temporary under dashboards and not under my home Organization => Organizations Cluster => Clusters

fakhrihawari commented 3 years ago

Organization => Organizations Cluster => Clusters what is that ? filter?

drfaustusfade commented 3 years ago

that's spelling correction to use the plural form in title and table column name

fakhrihawari commented 3 years ago
Screen Shot 2021-01-15 at 16 32 31

and the download is csv file

drfaustusfade commented 3 years ago

ok, well done

fakhrihawari commented 3 years ago

okay I will make PR

link https://github.com/drfaustusfade/ngm-reportEngine/pull/16 https://github.com/drfaustusfade/ngm-reportHub/pull/65

drfaustusfade commented 3 years ago

I think would be also nice to have type in quick table filtering for organization list and maybe to try having 100 list elements rather than 10

fakhrihawari commented 3 years ago

100 is too long, i propose 20 or 25

drfaustusfade commented 3 years ago

alright

fakhrihawari commented 3 years ago

Update: https://github.com/drfaustusfade/ngm-reportHub/pull/72

drfaustusfade commented 3 years ago

thanks on /organization-contact/... there was an user error TypeError: Cannot read property 'roles' of undefined

TypeError: Cannot read property 'roles' of undefined undefined
logger: console
arguments: [{"message":"Cannot read property 'roles' of undefined","name":"TypeError","stack":"TypeError: Cannot read property 'roles' of undefined\n    at Object.canEdit...
fakhrihawari commented 3 years ago

Ok, already fix it: https://github.com/drfaustusfade/ngm-reportHub/pull/72/files