Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Exile support #23

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

programmerjake
Copy link
Contributor

@programmerjake programmerjake commented Mar 23, 2024

progress:

  • mostly working
  • needs balancing (e.g. some items may have emc values greater than their craft results)
  • need to find all places where exchangeclone assumes mcl/mtg are the only options
  • some of the special functions crash (e.g. hammers right-click mining)
  • go through the list of exile items and assigning emc values to them all

basically everything in ExchangeClone is accessed through the Philosophers Stone (left-click while holding it opens a crafting interface), which you can find waay underground in chests.

Fixes: #22

Depends on https://codeberg.org/Mantar/Exile/pulls/779 (merged)

@ThePython10110
Copy link
Owner

Just so you know, pretty much everything is complete in the dev branch except the Zero Ring and Ring of Ignition.

@programmerjake
Copy link
Contributor Author

Just so you know, pretty much everything is complete in the dev branch except the Zero Ring and Ring of Ignition.

yup, i'll keep working on this, don't block releasing new stuff on me...it may be a while.

@programmerjake programmerjake force-pushed the exile-support branch 3 times, most recently from 3e01fb9 to 6de3aac Compare March 24, 2024 10:03
@programmerjake
Copy link
Contributor Author

I think this is now in a state where you can try it out if you like...though it's definitely WIP

@programmerjake programmerjake marked this pull request as ready for review May 6, 2024 15:53
@programmerjake
Copy link
Contributor Author

programmerjake commented May 6, 2024

I'm marking this as ready because it mostly works and because I'm not likely to do much more work on it in the near future (month or two).

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