Skip to content

users: correct achievements lookup for parent planet users (fixes #10275) - #10274

Open
Mutugiii wants to merge 1 commit into
masterfrom
claude/parent-planet-user-lookup-yqkgpl
Open

users: correct achievements lookup for parent planet users (fixes #10275)#10274
Mutugiii wants to merge 1 commit into
masterfrom
claude/parent-planet-user-lookup-yqkgpl

Conversation

@Mutugiii

@Mutugiii Mutugiii commented Aug 12, 2026

Copy link
Copy Markdown
Member

Fixes #10275

  • Resolves the achievements page user as local, parent, or child instead of treating every non-local user as a child
  • Loads parent planet users from parent_users/org.couchdb.user:<name> instead of child_users/<name>@<planetCode>
  • Treats a missing planet code as local before comparing it with configuration.parentCode
  • Adds unit tests for all three relationships

Avatar loading and achievement document IDs are unchanged.

https://claude.ai/code/session_017bvBBwBKLPRxMci11ctVqk

The achievements page treated every non-local user as a child user, so
users on the parent planet were looked up in child_users with an
<name>@<planetCode> id instead of parent_users with an
org.couchdb.user:<name> id, and their profile never loaded.

Resolve the relationship first (missing planet code counts as local
before comparing against configuration.parentCode), then pick the
database and document id from it. Achievement document ids and avatar
loading are unchanged.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017bvBBwBKLPRxMci11ctVqk
@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@Mutugiii, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 49 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
鈿欙笍 Run configuration

Configuration used: Repository UI

Review profile: QUIET

Plan: Pro Plus

Run ID: 9c45d061-cf48-4d5d-8cb0-71617e9a96c5

馃摜 Commits

Reviewing files that changed from the base of the PR and between 6c3ea09 and 687cc80.

馃搾 Files selected for processing (2)
  • src/app/users/users-achievements/users-achievements.component.spec.ts
  • src/app/users/users-achievements/users-achievements.component.ts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

鉂わ笍 Share

Comment @coderabbitai help to get the list of available commands.

@Mutugiii Mutugiii changed the title Refactor user relationship logic in UsersAchievementsComponent users: correct achievements lookup for parent planet users (fixes #10275) Aug 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Achievements: parent planet users are looked up in child_users

2 participants