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

BungeeCord bossbar support #155

Merged
merged 5 commits into from
Jun 28, 2024
Merged

Conversation

bivashy
Copy link
Owner

@bivashy bivashy commented Feb 17, 2024

Pull Request Etiquette

  • I have checked the PRs for upcoming features/bug fixes.

Changes

  • Internal code
  • API (affecting end-user code)
  • Other: BungeeCord and Velocity platform

Closes Issue: #154

This Pull Request replaces usage of native BossBar with Adventure API BossBar

@bivashy bivashy added the status: needs testing Issue or PR should be tested label Feb 17, 2024
@bivashy
Copy link
Owner Author

bivashy commented Feb 17, 2024

Current state: This implementation doesn't work quite well
Adventure-platform implementation have same problem:

java.lang.NoSuchMethodError: 'void net.md_5.bungee.protocol.packet.BossBar.setTitle(java.lang.String)'
    at com.bivashy.auth.lib.net.kyori.adventure.platform.bungeecord.BungeeFacet$BossBar.bossBarNameChanged(BungeeFacet.java:258)
    at com.bivashy.auth.lib.net.kyori.adventure.platform.facet.Facet$BossBar.bossBarInitialized(Facet.java:525)
    at com.bivashy.auth.lib.net.kyori.adventure.platform.bungeecord.BungeeFacet$BossBar.bossBarInitialized(BungeeFacet.java:250)

@bivashy
Copy link
Owner Author

bivashy commented Mar 8, 2024

Tested with new Adventure version that fixes this problem, everything works as expected

@bivashy bivashy marked this pull request as ready for review June 28, 2024 18:04
@bivashy bivashy removed the status: needs testing Issue or PR should be tested label Jun 28, 2024
@bivashy bivashy merged commit b6086a6 into main Jun 28, 2024
3 checks passed
@bivashy bivashy deleted the fix/154-bungeecord-bossbar-support branch July 3, 2024 14:10
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.

1 participant