-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
[Bug] Dropped accesories on death - Incompatiblity with Revive #78
Comments
Odd question but did you have any item within the cosmetic slots of Accessories? |
Nope, I did´t have any cosmetics on |
Within testing with the latest release of accessories and the compat layer everything but the cosmetic accessories are put within the grave. |
Well, strangely such is not dropping within the prod with the example mods but could come down to some weird incompat with Universal graves. |
Submit a pr to backport compat for Universal graves that may get pulled in hopefully fixing the issue fully. |
Thank you, hopefully it will have a more predictable behaviour, because as of right now, some accessories go into the grave and others don´t |
After updating to the latest universal grave version, 3.0.2, keeps happening |
@salva09 You could give You're in Grave Danger a try as latest 1.20.1 release added Accessories support: https://www.curseforge.com/minecraft/mc-mods/youre-in-grave-danger/files/5606122 |
I just found out the mod to blame is Revive by GobloxZ |
Honestly seems more like an incompatibility on their side. Currently no public API to resolve any problems and would recommend making a issue report on there side about this. |
Has nothing to do with Revive nor Fabric. I can reproduce the issue on Forge 47.3.0 on 1.20.1 with the only mods+their dependencies being: Accessories 1.0.0-beta26 + Curios Compat Layer 7.4.0 (It also happens with Quark's Totem of Holding but I didn't test that individually with other mods, only in my large modpack, so I don't include it for sake of integrity of the testing) The above mods result in accessories being flung out of the inventory upon death and never put into the gravestone/corpse. It looks like this is the exact same issue so i left a comment here, but will open a full issue report if requested. |
Such would be a different issue as it is a different set of mods altogether. Furthermore, this issue on Gravestones indicates that it will not support Curios when handling its inventory meaning it won't handle Accessories. A similar problem with Corpse as this issue also indicates the same thing. |
An issue should be made on Revive to either add support for Accessories or add API to do such but I would instead recommend just using supported grave mods like YIGD or Universal Graves |
Update on this: 53f4f9f should fix incompatibility for mods that don't add direct support for accessories like the ones mentioned above. |
Cool, I'll test this out when it becomes available, thanks! |
Fixed within 1.21 1.1.0 version for both before and fabric |
Info
Minecraft version: 1.20.1
Mod loader: Fabric
Accessories version: 1.0.0-beta.26
Accessories trinkets compat version: 3.8.5-beta.5
OS: Fedora Linux 40
Java runtime: OpenJDK 17.0.11
Minecraft launcher: Prism Launcher 8.4
Description
When playing with the accessories mod and gravestone mods, some accessories don´t get into the gravestone, and instead are dropped in place like vanilla minecraft.
This happenes for me with most accessories except for those in the spell book slot from the spell engine, bit happened with other accessoires like jewelry and backpacks
Mods tested
Gravestones:
Accessories:
Screenshots
Logs
The text was updated successfully, but these errors were encountered: