Skip to content

monthly active user count 추가 #420

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

Merged
merged 3 commits into from
Jun 10, 2024
Merged

monthly active user count 추가 #420

merged 3 commits into from
Jun 10, 2024

Conversation

jpham005
Copy link
Member

@jpham005 jpham005 commented Jun 9, 2024

실제로 활동중인 유저 수를 db collection 으로 따로 생성 했습니다.
scale team 참여자, logtime 보유자 기준 집계 입니다.

기존 작업중인 내역 때문에 main 에 바로 merge 합니다.

niamu01 and others added 2 commits April 9, 2024 19:21
이전에 수정했었는데 다른 곳에서도 발생하고 있어 추가로 넣었습니다.

- close #408
@jpham005 jpham005 added the enhancement New feature or request label Jun 9, 2024
@jpham005 jpham005 requested a review from niamu01 June 9, 2024 14:51
@jpham005 jpham005 self-assigned this Jun 9, 2024
@jpham005 jpham005 changed the base branch from dev to main June 9, 2024 14:51
Copy link
Contributor

@niamu01 niamu01 left a comment

Choose a reason for hiding this comment

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

제가 dev를 막아버린 형태가 되었군요... follow 관련 기능은 배포하지 않기로 결정한 기능인 만큼 브랜치로 다시 잘 빼보겠습니다.

올려주신 코드는 간단한 추가라 빠르게 확인했습니다.

이전에 프론트에서 하기로 했던 것으로 기억하지만, 백엔드에서 처리하는게 나을 것 같아 수정 합니다.
@jpham005 jpham005 merged commit fa015a0 into main Jun 10, 2024
@jpham005 jpham005 deleted the release/10.3 branch June 10, 2024 00:51
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
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants