Skip to content

Fix profile reply stats display#2742

Draft
guaka wants to merge 8 commits into
mainfrom
cursor/fix-profile-reply-stats
Draft

Fix profile reply stats display#2742
guaka wants to merge 8 commits into
mainfrom
cursor/fix-profile-reply-stats

Conversation

@guaka

@guaka guaka commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • fix the profile sidebar reply stats instead of removing reply rate
  • render reply time as the formatted duration returned by the API, for example 3 hours
  • keep displaying reply rate, including 0%, and add focused component coverage
  • merge current main so the branch has no conflicts

Validation

  • npm run test:client -- modules/users/tests/client/components/ProfileViewBasics.tests.js
  • ./node_modules/.bin/eslint modules/users/client/components/ProfileViewBasics.js modules/users/tests/client/components/ProfileViewBasics.tests.js
  • git diff --check

Notes

  • Replaces Fix profile reply stats display #2727 so the PR uses the correctly named branch: cursor/fix-profile-reply-stats.
  • Existing unrelated commits/files remain from the original branch history, including CI workflow updates, local Docker config, and signup test updates.

K added 8 commits April 28, 2026 11:30
…cursor/fix-profile-reply-stats

# Conflicts:
#	.github/workflows/lint.yml
#	.github/workflows/test.yml
#	deploy/docker/docker-compose.yml
#	modules/core/server/routes/core.server.routes.js
#	modules/experiences/tests/client/components/CreateExperience.client.component.tests.js
#	modules/tribes/tests/client/components/TribesPage.component.tests.js
#	modules/users/tests/client/components/ProfileViewBasics.tests.js
#	modules/users/tests/server/user-signup.server.routes.tests.js
@guaka guaka added the run-e2e add this label to a PR if you want the e2e tests to run label Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-e2e add this label to a PR if you want the e2e tests to run

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant