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

EssenceShopPrice feature #737

Merged
merged 6 commits into from
Jul 18, 2024

Conversation

UpFault
Copy link
Contributor

@UpFault UpFault commented May 28, 2024

added option to view how much essence will cost from bazaar in lore of Essence Shop Items

@LifeIsAParadox LifeIsAParadox added the reviews needed This PR needs reviews label May 28, 2024
@kevinthegreat1
Copy link
Collaborator

Thanks for your great pr’s. As you may see, we currently have quite a few pr’s awaiting review so this might take a while.

@kevinthegreat1 kevinthegreat1 added the new feature This issue or PR is a new feature label May 28, 2024
@UpFault
Copy link
Contributor Author

UpFault commented May 28, 2024

Thanks for your great pr’s. As you may see, we currently have quite a few pr’s awaiting review so this might take a while.

no worries

@UpFault
Copy link
Contributor Author

UpFault commented May 29, 2024

ZkWnrwz
fC9jrWU
UjRQHSC

Figured I'd drop these here

@UpFault
Copy link
Contributor Author

UpFault commented May 29, 2024

Also I should probably move the config value over to Item Tooltip in the general category rather than UI & Visuals(don't know what I was thinking here.)

@Fluboxer
Copy link
Contributor

May as well take it to your own visitor pr so they have a child - if you lack essence to buy thing, right clicking would just open bz with needed essence in your clipboard

@kevinthegreat1 kevinthegreat1 added the merge conflicts This PR has merge conflicts that need solving. label May 31, 2024
@LifeIsAParadox LifeIsAParadox removed the merge conflicts This PR has merge conflicts that need solving. label Jun 2, 2024
@kevinthegreat1

This comment was marked as resolved.

@kevinthegreat1 kevinthegreat1 mentioned this pull request Jun 2, 2024
@kevinthegreat1 kevinthegreat1 added changes requested This PR need changes and removed reviews needed This PR needs reviews labels Jun 2, 2024
@LifeIsAParadox LifeIsAParadox added merge conflicts This PR has merge conflicts that need solving. reviews needed This PR needs reviews and removed changes requested This PR need changes merge conflicts This PR has merge conflicts that need solving. labels Jun 2, 2024
@kevinthegreat1 kevinthegreat1 added the merge conflicts This PR has merge conflicts that need solving. label Jun 10, 2024
@LifeIsAParadox LifeIsAParadox removed the reviews needed This PR needs reviews label Jun 30, 2024
@LifeIsAParadox LifeIsAParadox added reviews needed This PR needs reviews and removed merge conflicts This PR has merge conflicts that need solving. labels Jul 14, 2024
@LifeIsAParadox LifeIsAParadox added changes requested This PR need changes and removed reviews needed This PR needs reviews labels Jul 18, 2024
@LifeIsAParadox LifeIsAParadox added reviews needed This PR needs reviews and removed changes requested This PR need changes labels Jul 18, 2024
kevinthegreat1
kevinthegreat1 previously approved these changes Jul 18, 2024
@LifeIsAParadox LifeIsAParadox added merge me please Pull requests that are ready to merge and removed reviews needed This PR needs reviews labels Jul 18, 2024
@LifeIsAParadox LifeIsAParadox added reviews needed This PR needs reviews and removed merge me please Pull requests that are ready to merge labels Jul 18, 2024
@LifeIsAParadox LifeIsAParadox added merge me please Pull requests that are ready to merge and removed reviews needed This PR needs reviews labels Jul 18, 2024
@kevinthegreat1 kevinthegreat1 merged commit cceaa38 into SkyblockerMod:master Jul 18, 2024
1 check passed
@LifeIsAParadox LifeIsAParadox removed the merge me please Pull requests that are ready to merge label Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature This issue or PR is a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants