Skip to content

Feature/achievements#224

Merged
bristermitten merged 16 commits into
TheDeveloperDen:masterfrom
Pdzly:feature/achievements
Feb 8, 2026
Merged

Feature/achievements#224
bristermitten merged 16 commits into
TheDeveloperDen:masterfrom
Pdzly:feature/achievements

Conversation

@Pdzly

@Pdzly Pdzly commented Jan 23, 2026

Copy link
Copy Markdown
Contributor

Add achievements

Pdzly and others added 9 commits November 12, 2025 23:18
…ions, and services.

- Introduce `/achievements` command for users to view their progress.
- Configure `notificationMode` and fallback channels in config files.
- Integrate achievement checks and notifications in message XP and bump listeners.
- Add database schema updates for tracking user achievements.
…g of inactive achievements.

- Implement `getActiveAchievements` and `getActiveAchievementsByTrigger` helper functions.
- Update services to use active achievements only, ensuring inactive achievements are excluded from processing.
- Introduce grouping of achievements by notification mode in notifications.
- Add `SPECIAL` achievement category and manual grant mechanism.
- Implement command for manually granting achievements.
- Update `/achievements` command to include `SPECIAL` category.
@Pdzly
Pdzly requested a review from bristermitten January 23, 2026 19:09
@Pdzly Pdzly assigned Pdzly and unassigned Pdzly Jan 23, 2026
@Pdzly Pdzly added the enhancement New feature or request label Jan 23, 2026
Pdzly added 4 commits January 23, 2026 20:19
…chievements to users.

- Restrict use to staff members only.
# Conflicts:
#	src/config.type.ts
#	src/index.ts
#	src/store/models/DDUser.ts
#	src/store/storage.ts
…nts from users.

- Restrict use to staff members only.
…entation, alignment).

- Improve readability for test files and module configuration.
- Minor updates to align code style with existing conventions.
- No functional changes were made.

@bristermitten bristermitten left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great start but needs some work

Comment thread src/modules/achievements/achievementDefinitions.ts Outdated
Comment thread src/modules/xp/dailyReward.command.ts Outdated
Comment thread src/modules/achievements/achievementNotifier.ts Outdated
Comment thread src/modules/achievements/achievementNotifier.ts Outdated
Comment thread src/modules/achievements/achievementService.ts
Comment thread src/modules/achievements/achievementService.ts Outdated
Pdzly added 3 commits January 26, 2026 12:45
…ling in notifications.

- Remove outdated "Monthly Champion" achievement definition.
- Use `highestDailyStreak` when checking achievements in daily reward flow.
… INTEGER` type.

- Update database schema and documentation to reflect changes.
- Remove unused `generateAchievementRecordId` helper method.
- Simplify achievement creation logic in `achievementService`.
…` helper method.

- Replace redundant mode-based switch statements with `sendByMode` calls.
- Remove unused `fakeMention` import.
@Pdzly
Pdzly requested a review from bristermitten January 30, 2026 17:05

@bristermitten bristermitten left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's do it

@bristermitten
bristermitten merged commit 04f2682 into TheDeveloperDen:master Feb 8, 2026
8 checks passed
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

None yet

Development

Successfully merging this pull request may close these issues.

2 participants