Skip to content

Add counter of active users to admin dashboard#5907

Merged
tramuntanal merged 23 commits intodecidim:developfrom
CodiTramuntana:feat/active_users_counter
May 5, 2020
Merged

Add counter of active users to admin dashboard#5907
tramuntanal merged 23 commits intodecidim:developfrom
CodiTramuntana:feat/active_users_counter

Conversation

@Leusev
Copy link
Copy Markdown
Contributor

@Leusev Leusev commented Apr 1, 2020

🎩 What? Why?

Adds an Activity dash into general admin dashboard with a counter of active users in Decidim's platform, making a distinction between whether they are admins or participants and three distinct periods: last 24h, last week and last month.

Closes #5780

📌 Related Issues

📋 Subtasks

  • Add CHANGELOG entry
  • Add documentation regarding the feature
  • Add/modify seeds
  • Add tests
  • Another subtask

📷 Screenshots (optional)

Before

image

After

image

@Leusev Leusev changed the title Counter of active users WIP : Counter of active users Apr 1, 2020
@Leusev Leusev removed the status: WIP label Apr 6, 2020
@Leusev Leusev changed the title WIP : Counter of active users Counter of active users Apr 6, 2020
@Leusev Leusev marked this pull request as ready for review April 6, 2020 11:26
@Leusev Leusev requested review from jesusdb and tramuntanal April 9, 2020 10:05
jesusdb
jesusdb previously approved these changes Apr 22, 2020
Copy link
Copy Markdown
Contributor

@jesusdb jesusdb left a comment

Choose a reason for hiding this comment

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

Great job, @Leusev!

@mrcasals
Copy link
Copy Markdown
Contributor

Hi @decidim/core, what's the status of this PR? We'll start working on #5725 and we might get some conflicts with this PR... 😄

@tramuntanal
Copy link
Copy Markdown
Contributor

I saw your PR @mrcasals when you get tests passing it'll be merged. We're not going to block, first inn wins 😉

@tramuntanal tramuntanal reopened this Apr 29, 2020
@tramuntanal
Copy link
Copy Markdown
Contributor

I closed by mistake.

@tramuntanal tramuntanal changed the title Counter of active users Add counter of active users to admin dashboard Apr 29, 2020
@Leusev Leusev requested review from jesusdb and mrcasals April 30, 2020 12:21
@mrcasals mrcasals removed their request for review May 4, 2020 09:11
ivan-mr
ivan-mr previously approved these changes May 5, 2020
Copy link
Copy Markdown
Contributor

@ivan-mr ivan-mr 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

@Leusev Leusev requested a review from ivan-mr May 5, 2020 12:21
Copy link
Copy Markdown
Contributor

@ivan-mr ivan-mr left a comment

Choose a reason for hiding this comment

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

Everything seems to be ok

@tramuntanal tramuntanal closed this May 5, 2020
@tramuntanal tramuntanal reopened this May 5, 2020
@tramuntanal tramuntanal merged commit 7e45b9b into decidim:develop May 5, 2020
@tramuntanal tramuntanal deleted the feat/active_users_counter branch May 5, 2020 13:51
ace pushed a commit to aspgems/decidim that referenced this pull request May 5, 2020
* develop: (29 commits)
  Update Conversations design with decidim-design UI (decidim#6008)
  Add counter of active users to admin dashboard (decidim#5907)
  Show activity graphs on admin dashboard (decidim#6030)
  Update sassc gem version (decidim#6062)
  Fix generator Gemfile after puma upgrade (decidim#6060)
  New Crowdin translations (decidim#6059)
  Add Slovak as a new language (decidim#6039)
  Remove all tests for i18n PRs (decidim#6061)
  Update move up and down buttons after dragging questions when managing questionnaire (decidim#5947)
  Fix using Decidim as a provider for omniauth authentication (decidim#6042)
  Add redesign for responsive public profile navigation tabs (decidim#6032)
  Add versioning pages to initiatives (decidim#5935)
  Notify users when roles get assigned (decidim#5886)
  Improve the budget page and the project card (decidim#5809)
  New Crowdin translations (decidim#6050)
  New Crowdin translations (decidim#6046)
  Ignore jobs on locales branches (decidim#6047)
  Automatic task for deleting Meeting Inscription data (decidim#5989)
  New Crowdin translations (decidim#5877)
  Ignore builds on Crowdin PRs (decidim#6037)
  ...
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

Successfully merging this pull request may close these issues.

Counter of active users

5 participants