Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix: use unique workspace owners over unique users#11044

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
f0ssel merged 7 commits intomainfromf0ssel/template-daus
Dec 7, 2023

Conversation

f0ssel
Copy link
Contributor

@f0sself0ssel commentedDec 5, 2023
edited
Loading

Closes#10820

Currently theactive_user_count on each template is not really "active users" but the count of any user that has made a workspace that was healthy based on the template.

This changes it to use the count of unique workspace owners based off the template, which I believe is what users will expect the number to be based on the issue made for this.

@f0sself0ssel closed thisDec 5, 2023
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsDec 5, 2023
@f0sself0ssel reopened thisDec 6, 2023
@f0sself0ssel changed the titlefix: use template daus over unique usersfix: use unique workspace owners over unique usersDec 6, 2023
@f0sself0ssel marked this pull request as ready for reviewDecember 6, 2023 16:22
Copy link
Member

@EmyrkEmyrk left a comment

Choose a reason for hiding this comment

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

👍 The original value is still there too if you want to pull the total ever. Reading the issue ticket, this all makes sense.

@f0ssel
Copy link
ContributorAuthor

Yeah I have a feeling people will want to know the total unique users that have ever used the templatesomewhere in the UI so I wanted to leave it for now.

@f0sself0ssel merged commit8aea604 intomainDec 7, 2023
@f0sself0ssel deleted the f0ssel/template-daus branchDecember 7, 2023 15:53
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers

@kylecarbskylecarbskylecarbs left review comments

@EmyrkEmyrkEmyrk approved these changes

@deansheatherdeansheatherAwaiting requested review from deansheather

Assignees

@f0sself0ssel

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Used by in templates list is innacurate
3 participants
@f0ssel@Emyrk@kylecarbs

[8]ページ先頭

©2009-2025 Movatter.jp