Skip to content

Update AHelpUIController.cs#12

Open
Punkercelestial wants to merge 1 commit into
Project-Dumont:masterfrom
Punkercelestial:Ahelp-update
Open

Update AHelpUIController.cs#12
Punkercelestial wants to merge 1 commit into
Project-Dumont:masterfrom
Punkercelestial:Ahelp-update

Conversation

@Punkercelestial

Copy link
Copy Markdown

Breve Descrição

Adicionados dicionários/lista para armazenar mensagens recebidas antes do carregamento do histórico, por canal.
Novo método LoadHistory que insere primeiro as mensagens do histórico e, em seguida, as mensagens do buffer, preservando a ordem cronológica.
Controle por canal (_historyLoaded) para decidir se novas mensagens vão para o buffer ou são exibidas imediatamente.
O método Clean agora só limpa a exibição visual, sem afetar o buffer ou a flag de histórico.
No OnBwoinkTextHistoryMessage, a chamada a Clean foi eliminada, evitando perda de mensagens antes de aplicar o histórico.

ESSA PR NAO FOI TESTADA INGAME(POR ISSO NAO É UMA CORREÇÃO DEFINITIVA), DEVIDO COMPLICAÇÕES TECNICA DA AUTORA, RECOMENDA-SE FORTEMENTE CHERRY-PICK PARA TESTE ANTES

Tenta corrigir o desync do ahelp
@Kyoth25f

Copy link
Copy Markdown
Contributor

Estou um pouco perdido com a mudança. Você está tentando resolver um problema? Se sim, qual? Você está implementando uma nova funcionalidade? Se sim, qual?

Você deu uma descrição puramente técnica, é bom colocar qual o seu objetivo. Ele me da um norte pra ver se o que você escreveu realmente cumpre com o que você propos. E, também, me permite te ajudar a pensar se essa é a melhor forma de fazer isso.

Eu acredito que você está implementando uma nova funcionalidade de histórico de ahelps. Mas, se não estou enganado, esse código tem exatamente essa funcionalidade e é o Gaby que não. Talvez eu esteja fazendo confusão com algo aqui.

ESSA PR NAO FOI TESTADA INGAME(POR ISSO NAO É UMA CORREÇÃO DEFINITIVA), DEVIDO COMPLICAÇÕES TECNICA DA AUTORA, RECOMENDA-SE FORTEMENTE CHERRY-PICK PARA TESTE ANTES

É bemmm, tipo MUITO, interessante que você teste as suas mudanças. Eu não sei qual o seu objetivo e também não sei o que testar. Se tem alguém melhor para testar as suas mudanças, essa pessoa é você.

Eu rodo todos os PRs e reviso os códigos, mas só você é capaz de me dizer se o que está acontecendo é o que você quer que aconteça. Pode acontecer do código fazer sentido e não ser o que você espere.

@Punkercelestial

Copy link
Copy Markdown
Author

Compreendo sua duvida, pode deixar a pr em open por enquanto, quando eu ter tempo suficiente que não seja nas minhas madrugadas irei testar devidamente o codigo.
Sobre a intenção é para correção de desync na ui do ahelp, não puxei a do gaby pq sao tipos diferentes.
sob os detalhes meramente tecnicos, perdão, mas atraves deles é bem facil identificar inclusive a intenção das mudanças.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants