Skip to content

[CHANGE] Refonte des hologrammes Paper#1344

Closed
ar1hurgit wants to merge 12 commits into
ServerOpenMC:masterfrom
ar1hurgit:hologram
Closed

[CHANGE] Refonte des hologrammes Paper#1344
ar1hurgit wants to merge 12 commits into
ServerOpenMC:masterfrom
ar1hurgit:hologram

Conversation

@ar1hurgit

@ar1hurgit ar1hurgit commented Jul 12, 2026

Copy link
Copy Markdown

Petit résumé de la PR:

Cette PR conserve les hologrammes virtuels NMS et corrige leur cycle de viewers. Les chunks ne sont pas forcés à rester chargés.

Étape nécessaire afin que la PR soit fini (si PR en draft)

  • Suivre le Code de Conduite
  • Enlever tous les imports non utilisés
  • Bien documenter la feature
  • Fournir un profileur (si besoin/demandé par un admin)
  • Avoir une milestone associée à la PR
  • Valider tout les checks
  • Tester et valider la feature/changement

Decrivez vos changements

  • Les TextDisplay restent des hologrammes virtuels NMS envoyés par paquets aux joueurs proches.
  • La liste des viewers est de nouveau actualisée régulièrement : les hologrammes réapparaissent après une reconnexion.
  • /leaderboard setPos sauvegarde la position dans leaderboards.yml, recharge la configuration et envoie le déplacement de l’hologramme actuel aux viewers.
  • Les textes legacy des hologrammes sont désérialisés avant leur création pour supprimer les warnings Adventure concernés.

@iambibi

iambibi commented Jul 12, 2026

Copy link
Copy Markdown
Member

???? Pourquoi enlever les NMS et mettre des trucs paper

@iambibi

iambibi commented Jul 12, 2026

Copy link
Copy Markdown
Member

Faire que des chunks contenant des hologrammes soient tjr chargé n'est vraiment vraiment pas optimisé

@iambibi

iambibi commented Jul 12, 2026

Copy link
Copy Markdown
Member

Correction des bugs avec /setpos dit nous lesquels?

@iambibi

iambibi commented Jul 12, 2026

Copy link
Copy Markdown
Member

Rien que sur le résumé je suis pas d'accord et même garde les checks de base qui sont mis sur les PR au lieu de tt delete.
Donc je suis pas d'accord sur le principe de ta pr, je vais pas review

@ar1hurgit

Copy link
Copy Markdown
Author

oe si tu voulais pas changer et rester sur NMS le pr ne sera pas accepter

@ar1hurgit ar1hurgit closed this Jul 12, 2026
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