Conversation
- Add HousePoints, Achievement, Quest, Notification models - Add Player fields: EloRating, House, equipment slots, login streak, referral, onboarding - Add DTOs for House, Quest, Achievement, Notification, Equipment - Update PlayerProfileResponse with new fields and rank system - Update RegisterRequest with optional House parameter - Configure EF Core relationships in AppDbContext - Fix test files for updated RegisterRequest signature Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: jwraats <3438726+jwraats@users.noreply.github.com>
Implement services: HouseService, QuestService, AchievementService, NotificationService, EquipmentService, LoginRewardService. Implement controllers: HouseController, QuestController, AchievementController, NotificationController, EquipmentController, LoginRewardController. Update BattleService with ELO rating, equipment bonuses, house points, quest progress tracking, achievement checks, and battle notifications. Update AuthService with house selection and player referral rewards. Update PlayerService/Controller with battle stats endpoint. Register all new services in Program.cs DI container. Update BattleServiceTests to pass new dependencies as mocks. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: jwraats <3438726+jwraats@users.noreply.github.com>
- Fix streak calculation to single-pass in PlayerService - Use string.IsNullOrEmpty for house filtering in HouseService - Remove unused CalculateDamage method from BattleService - Move AwardHousePointsRequest to House DTOs file Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: jwraats <3438726+jwraats@users.noreply.github.com>
…improvement ideas Agent-Logs-Url: https://github.com/jwraats/Wizard-RPG/sessions/949b9574-732c-4274-8b78-d7d1a75e9090 Co-authored-by: jwraats <3438726+jwraats@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Implement improvements from documentation
Implement improvement ideas: ELO ranking, house system, equipment, quests, achievements, notifications, login rewards
Mar 31, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implements the non-game improvement ideas from
docs/IMPROVEMENT_IDEAS.md— ranking, houses, equipment, quests, achievements, notifications, login rewards, referral enhancements, and onboarding.Backend — New Models & Services
Player.EloRating): K=32 calculation on battle finish, 5 rank tiers (Apprentice 🪄 → Grand Wizard 👑)Housefield on Player,HousePointsmodel for activity tracking,HouseService/HouseControllerwith aggregated house leaderboardEquippedWandId, etc.),EquipmentServicevalidates item type → slot mapping,BattleService.CalculateDamageWithEquipmentAsyncapplies stat bonuses in combatQuestmodel withQuestType(Daily/Weekly),QuestServicegenerates templated quests, tracks progress on battle wins/bets/deposits, auto-awards rewardsfirst_blood,spell_master,gold_hoarder,social_butterfly,veteran,grand_wizard), checked after battlesNotificationmodel,NotificationServicewith unread count, used by battle/quest/achievement/referral flowsLastLoginDate/LoginStreak, escalating rewards (Day 1: 10g → Day 30: 500g)GetBattleStatsAsyncendpoint returning win/loss/damage/streak/most-used-spellBackend — Modified Files
BattleService: ELO calc, equipment bonuses, house points, quest progress, achievement checks, notifications on finishAuthService: House selection + player referral code lookup on registrationPlayerService: AddedBattleStatsResponse, rank tier/badge mapping in profile responseProgram.cs: 6 new scoped services registeredFrontend
QuestResponse,AchievementResponse,NotificationResponse,EquipmentSlots,HouseLeaderboardEntry,LoginRewardStatus,BattleStatsResponse), updatedRegisterRequestandPlayerProfileResponsehouse,quest,achievement,notification,equipment,loginReward)notification,quest,equipment,house)Not yet complete
ProfileView (equipment/achievements/stats sections), LeaderboardView (ELO/house columns), BankView (equip button), NavBar (notification bell), NotificationsView, HouseView, and router updates for new pages are not included in this PR.