Deprecate and remove findStateAtEvent method in StateService
@debdutdeb ci sta già lavorando.
Dal 7/10/2025.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
Description
The findStateAtEvent method in StateService is marked with a TODO comment indicating it should be removed.
Location: packages/federation-sdk/src/services/state.service.ts (around line 897)
Current State
The method is marked with // TODO: remove this and currently wraps getStateAtEvent.
Suggested Migration Path
- Add a JSDoc
@deprecatedannotation to the method pointing users togetStateAtEvent - Identify and migrate all callers to use
getStateAtEventdirectly - Remove commented-out tests in
state.service.spec.tsthat reference this TODO - Remove the method once all migrations are complete
Context
- PR: https://github.com/RocketChat/homeserver/pull/252
- Comment: https://github.com/RocketChat/homeserver/pull/252#discussion_r2410585212
- Requested by: @debdutdeb
- Lingua principale
- TypeScript
- Stelle
- 17
- Fork
- 18
- Merge medio
- 2h 11m
- PR unite (30g)
- 2
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di RocketChat/homeserver
-
pruneEventDict and signEvent's `prune` flag should go away entirely, replaced byForse di nuovo libera @sampaiodiego l’ha presa 76 giorni fa e non c’è nessuna pull request aperta. Apertatodo :spiral_notepad: type: feature
RocketChat/homeserver#411 · 1 commento · 1 assegnatario ·
-
Include banned and invited servers in event destination setForse già presa @ggazzo l’ha presa 233 giorni fa. Apertatype: bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 52/100
RocketChat/homeserver#388 ·
-
we need a map of rooms and destinations to avoid having to get rooms state just to send an event to all servers in the room.Forse di nuovo libera @sampaiodiego l’ha presa 249 giorni fa e non c’è nessuna pull request aperta. Apertatodo :spiral_notepad: type: feature
RocketChat/homeserver#342 · 4 commenti · 1 assegnatario ·
-
this may not be the best place to do this validationForse di nuovo libera @sampaiodiego l’ha presa 326 giorni fa e non c’è nessuna pull request aperta. Apertatodo :spiral_notepad:
RocketChat/homeserver#322 · 1 commento · 1 reazione · 1 assegnatario ·
-
check if all sendEventToAllServersInRoom should be followed by an emitterForse di nuovo libera @ggazzo l’ha presa 359 giorni fa e non c’è nessuna pull request aperta. Apertatodo :spiral_notepad:
RocketChat/homeserver#304 · 1 assegnatario ·
Tutte le issue di RocketChat/homeserver
Issue simili
-
Bug: Broken RTL / BiDi text rendering when mixing Persian (Farsi) with English or numbersForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertabug
Difficoltà 2/5 Mezza giornata Idoneità per principianti 68/100
amosblomqvist/learnharness#758 ·
I maintainer di solito rispondono entro 2 giorni
-
Change WorkspacesAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
bitfocus/companion-module-renewedvision-propresenter-api#134 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
rcpch/digital-growth-charts-react-component-library#304 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
dsh-market/dsh-market#1210 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
trustjonathan/STUDY-HUB#13 ·