Skip to content

feat: add super-saiyan achievement - #1127

Merged
paulinevos merged 1 commit into
dev-null-sh:productionfrom
paulinevos:feat/super-saiyan-achievement
Oct 24, 2026
Merged

paulinevos merged 1 commit into
dev-null-sh:productionfrom
paulinevos:feat/super-saiyan-achievement

Conversation

@paulinevos

Copy link
Copy Markdown
Contributor
image

Copilot AI balanced review requested due to automatic review settings October 24, 2026 18:52
@paulinevos
paulinevos merged commit 3a9a137 into dev-null-sh:production Oct 24, 2026
5 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused changes are consistent across calculation, rendering, documentation, and boundary tests, with no unresolved blocking findings.

No open issues 🎉

What changed in this PR

Adds the super-saiyan badge to the profile achievement system for users with at least 9,000 current followers, without awarding XP.

Changes:

  • Defines the badge and calculates capped progress from the current follower count.
  • Adds a Dragon Ball icon to the achievement shelf and dialog.
  • Updates documentation and tests for unlocking, relocking, progress limits, and unchanged XP.
File Description
tests/​Unit/​Queries/​UserAchievementsQueryTest.php Updates badge count and tests follower thresholds and XP behavior.
resources/​js/​components/​dragon-ball-icon.tsx Adds the badge’s SVG icon.
resources/​js/​components/​achievements-shelf.tsx Registers the icon in the achievement shelf.
resources/​js/​components/​achievements-dialog.tsx Registers the icon in the achievement dialog.
KNOWLEDGE_BASE.md Documents the fifteenth badge and its unlock requirement.
app/​Queries/​UserAchievementsQuery.php Calculates follower-based badge progress.
app/​Enums/​Achievement.php Defines the badge metadata and 9,000-follower target.

Review effort: Balanced

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.

3 participants