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

[KAPP-175] Staff Dashboards #353

Merged
merged 13 commits into from
Apr 8, 2019

Conversation

ouranos
Copy link
Contributor

@ouranos ouranos commented Mar 28, 2019

Summary

This feature allow staff to create their own dashboards to view their customer data from the admin panel.
The dashboard are owned by the staff but use the customer company as data source.

Technical details

The main difficulty was to have two different modes for Impac:

  • Dashboard designer when creating/editing templates
  • Normal mode when creating/editing staff dashboard

This is possible through the ImpacConfigSvc which reconfigures impac-angular at runtime (see maestrano/impac-angular#565)

Note that we use the same ImpacController to display templates and staff dashboard so this is controlled via state data.

On the UI side, there's a new mnoe-staff-dashboard-list component, which just display the list of staff dashboards.

impac-angular was updated to 1.8 to have the latest features (notably the templates).

I've noted that I'll need to implement the feature flag in the backend as well.

Screenshots

Staff Dashboard List

staff_dhb_list

Create a new staff dashboard

staff_dhb_create

Staff Dashboard view

staff_dhb_view

Copy link
Contributor

@cesar-tonnoir cesar-tonnoir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me / always a bit hard to review the HTML, but the angular code seems correct

src/app/impac.config.coffee Show resolved Hide resolved
src/app/services/impac-config/impac-config.coffee Outdated Show resolved Hide resolved
Required changes have been released in 1.8.5
@ouranos ouranos merged commit be2d67a into maestrano:master Apr 8, 2019
@ouranos ouranos deleted the feature/advisor-dashboard branch April 8, 2019 06:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants