Skip to content

[Fix] Repair levers opening the Evolving XP Transfer Window#4607

Merged
Akkadius merged 1 commit intoEQEmu:masterfrom
neckkola:FixEvolveItemsXPTransferWindow
Jan 23, 2025
Merged

[Fix] Repair levers opening the Evolving XP Transfer Window#4607
Akkadius merged 1 commit intoEQEmu:masterfrom
neckkola:FixEvolveItemsXPTransferWindow

Conversation

@neckkola
Copy link
Copy Markdown
Contributor

Description

Levers outside of Corathus were causing the Evolving Item XP Transfer Window to open, in error.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Testing

Tested by clicking a level in Nadox, no issue. Clicking on the correct lever in Corathus opened the window.

Clients tested:
RoF2

Checklist

  • I have tested my changes
  • I have performed a self-review of my code. Ensuring variables, functions and methods are named in a human-readable way, comments are added only where naming of variables, functions and methods can't give enough context.
  • I own the changes of my code and take responsibility for the potential issues that occur

…ndow would be opened, in error. It should only open in Corathus.
@Akkadius Akkadius merged commit e83d094 into EQEmu:master Jan 23, 2025
@Akkadius Akkadius mentioned this pull request Jan 27, 2025
mute pushed a commit to mute/Server that referenced this pull request Mar 3, 2025
Valorith pushed a commit to Valorith/Server that referenced this pull request Mar 22, 2026
47 test cases across 15 sections covering all evolution types (XP, kill
count, mob race, zone), activation/timer mechanics, evolution completion,
augment handling, XP transfer window, merchant/parcel interactions,
persistence, GM commands, edge cases, and regression tests for PRs EQEmu#4607,
EQEmu#4992, #8, and #98. Includes ready-to-run SQL setup for 5 test item
families with low thresholds for fast manual testing.

https://claude.ai/code/session_017wrKmjuMTicYybGCRiBBFG
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