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

Problema em pegar a agenda do dia com várias threads com o termo "quem viu reage com ..." #7

Open
Rod-Pinheiro opened this issue Apr 7, 2022 · 6 comments

Comments

@Rod-Pinheiro
Copy link

Olá pessoal, bom trabalho a todos os envolvidos na criação dessa extensão que pode ser de grande ajuda a nós estudantes!

Me deparei com um problema no uso da extensão, já que no dia de hoje houveram várias threads postadas com a expressão "quem viu reage com ...", o que impossibilita a extensão de criar a agenda, já que a thread selecionada pelo regex não corresponde a agenda.

Acredito que o problema possa ser resolvido com uma alteração do regex usado na função getLastScheduleDay().

Espero ter conseguido explicar o problema que tive.
Bom trabalho galera!!

@pedrobritobr
Copy link
Owner

Olá pessoal, bom trabalho a todos os envolvidos na criação dessa extensão que pode ser de grande ajuda a nós estudantes!

Me deparei com um problema no uso da extensão, já que no dia de hoje houveram várias threads postadas com a expressão "quem viu reage com ...", o que impossibilita a extensão de criar a agenda, já que a thread selecionada pelo regex não corresponde a agenda.

Acredito que o problema possa ser resolvido com uma alteração do regex usado na função getLastScheduleDay().

Espero ter conseguido explicar o problema que tive. Bom trabalho galera!!

Oi Rod, de qual turma você é? Estarei estudando um novo jeito pra capturara a agenda. Muito obrigado por testar a extensão, fiquei feliz demais por abrir uma issue.

@Rod-Pinheiro
Copy link
Author

Fala pedro! Sou da turma 14-A e vc? Quando der vou gerar um fork pra testar aqui se eu consigo um jeito tb, um dos problemas é que a thread não é totalmente padronizada entre todas as turmas.

@pedrobritobr
Copy link
Owner

Rod, eu fui ver a msg da turma, mexi no código, e deu bom aqui, quando puder, testa aí ^^

@pedrobritobr
Copy link
Owner

Já dei o push, as atualizações demoram cerca de 4 dias pra chegar na chrome store.

@Rod-Pinheiro
Copy link
Author

Boa! Vi que vc adicionou um regex lá pra buscar a string "fixado por". Ainda não atualizou a extensão então ainda não consegui testar. Não sei como é a mensagem na sua turma, mas deixo como recomendação tb fazer o regex de "agenda".
Vou deixar aqui embaixo um exemplo da agenda do dia da minha turma e como fica o elemento HTML

<pre class="c-mrkdwn__pre" data-stringify-type="pre">Agenda do dia 30.1 - MongoDB e arquitetura MSC</pre>

image

@pedrobritobr
Copy link
Owner

Bem apontado, talvez seja melhor usar a combinação de 'fixado por' e 'agenda' ao invés de um regex louco para pegar por 'reage com'. Vou trocar aqui

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

No branches or pull requests

2 participants