Skip to content

fix(raf): update same-IP text based on endRAFOnSameIP setting#182

Merged
Nyeriah merged 1 commit intoazerothcore:masterfrom
Nyeriah:fix/raf-same-ip-text
Mar 27, 2026
Merged

fix(raf): update same-IP text based on endRAFOnSameIP setting#182
Nyeriah merged 1 commit intoazerothcore:masterfrom
Nyeriah:fix/raf-same-ip-text

Conversation

@Nyeriah
Copy link
Copy Markdown
Member

@Nyeriah Nyeriah commented Mar 26, 2026

Summary

  • Add end_raf_on_same_ip setting to the RAF config defaults and admin panel dropdown
  • When endRAFOnSameIP is enabled: keep the existing warning that RAF will be automatically removed
  • When endRAFOnSameIP is disabled: show a more coherent message matching ChromieCraft's wording"Whilst you can recruit a friend who shares an IP address, you will only get the teleport & XP bonuses, no rewards will be given"
  • Wrap Eluna DB queries in a try/catch so the RAF page shows a clear "No RAF installation found" error instead of a blank page when the database is not configured

Test plan

  • Verify the admin panel shows the new "RAF: End RAF on same IP" dropdown under Eluna settings
  • Toggle end_raf_on_same_ip to Enabled and confirm the original warning text appears on the RAF page
  • Toggle end_raf_on_same_ip to Disabled and confirm the updated ChromieCraft-style text appears
  • Disconnect or misconfigure the Eluna database and verify the RAF page shows the friendly error message instead of a blank page

🤖 Generated with Claude Code

…cefully

When endRAFOnSameIP is disabled, show a more coherent message matching
ChromieCraft's wording. Wrap Eluna DB queries in try/catch so the RAF
page displays a clear error instead of a blank page when the database
is not configured.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Nyeriah Nyeriah force-pushed the fix/raf-same-ip-text branch from 0324430 to eb9150a Compare March 26, 2026 22:10
@Nyeriah Nyeriah merged commit 173426e into azerothcore:master Mar 27, 2026
1 check passed
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.

2 participants