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

Implement Resource page #17

Open
dhornbein opened this issue Jun 30, 2017 · 14 comments
Open

Implement Resource page #17

dhornbein opened this issue Jun 30, 2017 · 14 comments
Assignees

Comments

@dhornbein
Copy link

Resource page (Mockup here) contains a filterable map, list of active resources along the right, and a data list under the map.

Page Structure

<main role='main' class='main'>
  
  <div class="row">
    <div class="small-9">
      <div class='row'>
        <div class='medium-4 columns'>
          <!-- FILTER HERE -->
        </div>
        <div id='map-here' class='medium-8 columns'>
          <!-- MAP HERE -->
        </div>
      </div>
      <div class="row">
        <div class="medium-12 column">
          <!-- DATA LIST HERE -->
        </div>
      </div>
    </div>
    <div class="small-3">
      <div class="row">
        <div class="panel">
          <!-- ACTIVE RESOURCES HERE -->
        </div>
      </div>
    </div>
  </div>
  
</main>

Mockup

Direct link
report page mock

@flavour
Copy link
Member

flavour commented Jun 30, 2017

ok, so this is a global 'Resources Browse' page...is there a corresponding 'Resource profile' page?

@flavour
Copy link
Member

flavour commented Jun 30, 2017

Why are there 2x dataTables here? Why not have 1 which is filterable & has a column for 'Active'?
That seems easier for the users & faster: less to render, less to update via AJAX

@devinbalkind
Copy link
Member

The client explicitly asked for two views - one with a list of all active resources and another where the user can view everything.

The table columns for the area below the map are here: https://airtable.com/tblHmfKKOGh4uzrHi/viwd73fBfzjtqBkFJ

"Updates" would link to any updates that come from that Resource. It could be a number showing the quantity of updates from that Resource - and on click reveals a popup table with an update log: Date, Title, Priority, Incident, Event. This means we should also add a Resource field to the Updates data model.

As for a Resource profile, we're displaying all the resource data we have in the main table + the updates popup.

So, for the right column, when clicking on the Resource Group/Name, I think it could just display the same popup update log as when the user clicks on the Updates quantity in the main table.

That seems like the simplest/easiest answer to me.

Let me know what you think.

@flavour
Copy link
Member

flavour commented Jul 4, 2017

Do we not show the Alerts & Events panels on this page?

@flavour
Copy link
Member

flavour commented Jul 4, 2017

No create button/form anywhere on this page?
(I know that WACOP imports all units from ADASHI, but my understanding was that we were building to allow standalone use)

@flavour
Copy link
Member

flavour commented Jul 4, 2017

You show a column 'Organizations' in the airtable.
ADASHI-imported Units have no Organization.
Beyond that we do have a data model to link Groups with Organizations, but normally this would be 1-1.
What is the usecase for 1 Group belonging to multiple Orgs?
(No huge issue here...just checking whether we make this 1-1 (simpler UI for users) or we need the M2M)

@flavour
Copy link
Member

flavour commented Jul 4, 2017

I pushed a 1st version & updated Demo
I had to add 'columns' to the small-3/small-9 to get the active units to display on the right-hand side
Still needs a little CSS tweaking

Note that there are 2 different Status fields for Resources, each with their own options.
One is for the Unit itself: I use tis one in the all resources dataTable at the bottom.
One is for the status within an Incident: I use this for the Active Incidents.

Currently the 'Current Incident' column shows all Incidents that a Unit is linked to....we need to think through how to make this just the 'Current' one...perhaps we need an additional flag to show whether a unit is actie on an Incident & only allow that to be on 1 Incident at a time.
This could be separate to the Status...or linked to 1 or more of the per-Incident Statuses.
Of course this is harder to manage in the free-for-all of the ADASHI world as they can invent new statuses for us which we wouldn't be able to set this flag correctly for without manual intervention.

@flavour
Copy link
Member

flavour commented Jul 4, 2017

As for a Resource profile, we're displaying all the resource data we have in the main table + the updates popup.

ok...but then how would the data be edited?

So, for the right column, when clicking on the Resource Group/Name, I think it could just display the same popup update log as when the user clicks on the Updates quantity in the main table.

If we ignore the edit-ability, then what about the bottom dataTable? Same thing?

@devinbalkind
Copy link
Member

  1. Do we not show the Alerts & Events panels on this page?
    Yes let's have them for now.

No create button/form anywhere on this page?
(I know that WACOP imports all units from ADASHI, but my understanding was that we were building to allow standalone use)

Lets keep it off for now. We can always add a create button later.

You show a column 'Organizations' in the airtable.
ADASHI-imported Units have no Organization.
Beyond that we do have a data model to link Groups with Organizations, but normally this would be 1-1.
What is the usecase for 1 Group belonging to multiple Orgs?
(No huge issue here...just checking whether we make this 1-1 (simpler UI for users) or we need the M2M)

Without Organizations being associated with Resources, it's hard to imagine how resource info would be useful at all. So I think it's good to keep that field there. I think 1 to 1 relationship works well for this.

Currently the 'Current Incident' column shows all Incidents that a Unit is linked to....we need to think through how to make this just the 'Current' one...perhaps we need an additional flag to show whether a unit is active on an Incident & only allow that to be on 1 Incident at a time.
This could be separate to the Status...or linked to 1 or more of the per-Incident Statuses.
Of course this is harder to manage in the free-for-all of the ADASHI world as they can invent new statuses for us which we wouldn't be able to set this flag correctly for without manual intervention.

Interesting. I didn't see the Resource Status data before (ex. QUARTERS). Did it just get revealed?

Do you think Resources should only be allowed to be active on one incident at a time? If so, can ADASHI data trigger effectively trigger that flag? Another way to go about this might just be to filter "Current Incident" data to just being Incidnets with the status "active". If a unit is connected to multiple Active Incidents, then maybe that's the information that should be conveyed.

As for a Resource profile, we're displaying all the resource data we have in the main table + the updates popup.

ok...but then how would the data be edited?

Maybe an edit button on the resource page (ex. http://cad.aidiq.com/eden/pr/group/251/read)?

So, for the right column, when clicking on the Resource Group/Name, I think it could just display the same popup update log as when the user clicks on the Updates quantity in the main table.
If we ignore the edit-ability, then what about the bottom dataTable? Same thing?

Maybe it's better if both go to the resource page (ex. http://cad.aidiq.com/eden/pr/group/251/read), where there would be an edit button as well as a tab for updates that show all the posts? What do you think of that?

@flavour
Copy link
Member

flavour commented Jul 7, 2017

Without Organizations being associated with Resources, it's hard to imagine how resource info would be useful at all. So I think it's good to keep that field there. I think 1 to 1 relationship works well for this.

OK, done...should we add this column to the Resopurces tables in the Event/Incident Profile pages too?
(Speaking of which, should we add the Incident column to this table in the Event Profile page?).

I didn't see the Resource Status data before (ex. QUARTERS). Did it just get revealed?

Yes, wasn't on any of the custom pages before

Do you think Resources should only be allowed to be active on one incident at a time?

I guess you could have a single Unit have a status on multiple Incidents, even if the status shouldn't really be able to be fully 'Active' on multiple (but could be 'Notified' or 'Not Available'.

If so, can ADASHI data trigger effectively trigger that flag?

Not reliably, no

Another way to go about this might just be to filter "Current Incident" data to just being Incidnets with the status "active".

Yes, showing just 'Not Closed' Incidents is a bigh step forward and easy, done.

If a unit is connected to multiple Active Incidents, then maybe that's the information that should be conveyed.

+1 - this is certainly the case with current ADASHI demo data

Maybe it's better if both go to the resource page (ex. http://cad.aidiq.com/eden/pr/group/251/read), where there would be an edit button

Agree this is better, done

as well as a tab for updates that show all the posts?

ok, will work on that now

@flavour
Copy link
Member

flavour commented Jul 7, 2017

as well as a tab for updates that show all the posts?

Should this be a dataList? (cards) [Not yet sure how hard this would be] or a Table?

@devinbalkind
Copy link
Member

devinbalkind commented Jul 7, 2017 via email

@flavour
Copy link
Member

flavour commented Jul 7, 2017

Is there any more data fields I should be aware of coming out of Adashi?

I don't think so

Can you email me that raw data so I can take a look?

https://github.com/sahana/eden/blob/master/modules/templates/WACOP/Demo/AVLExport.xml
https://github.com/sahana/eden/blob/master/modules/templates/WACOP/CadStatusConfig.xml

You can display this via table. :)

Which columns then?
dataList is actually done, although needs the design applying, so not hard & I think might be better?

@devinbalkind
Copy link
Member

Okay yes I think datalist is better for continuity etc.

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

No branches or pull requests

3 participants