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

CPS-472: Add My Activities Feed dashlet #754

Closed
wants to merge 4 commits into from

Conversation

reneolivo
Copy link
Member

Overview

This PR reintroduces the My Activities dashboard tab, but as a dashlet instead.

The My Activities tab was removed in #717

How it looks

pr

Technical Details

We created a new directive that has all the filters needed for the activity feed to display the incomplete activities belonging to the logged in user. This directive is used as the dashlet on the dashboard.

The skipUrlParamatersBinding parameter was added to the activity feed directive because the dashboard doesn't have the angular router enabled and binding URL parameters break. This only blocks the filters from being stored in the URL in case they refreshe it.

@reneolivo reneolivo added the enhancement New feature or request label Mar 30, 2021
@jamienovick
Copy link

Guys, this clearly doesn't fit in a dashlet and there is already a my activities dashlet so I'm not sure why we've gone down this route?

@reneolivo
Copy link
Member Author

Agreed @jamienovick . Can you please coordinate with @james-compucorp to get the right specs for this one?

The good news is that the dashlet can support Angular so we have a bit more flexibility in what we can provide.

@reneolivo
Copy link
Member Author

Closing until we agree on a solution.

@reneolivo reneolivo closed this Apr 13, 2021
@reneolivo reneolivo deleted the CPS-472-my-activities-feed branch April 13, 2021 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants