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

feat: add quiz event #83

Open
wants to merge 18 commits into
base: feat/quiz
Choose a base branch
from

Conversation

fernanduandrade
Copy link
Contributor

@fernanduandrade fernanduandrade commented Apr 23, 2023

@Novout PR draft do evento de coding para o bot
Primeiramente tinha feito o evento utilizando um json(event_coding.json
depois substitui por uma api generica para deixar mais dinamico

@Novout
Copy link
Member

Novout commented May 21, 2023

Agora com o firebase acho que dá para implementar isso. Teria como resolver os conflitos @fernanduandrade ?

@Novout Novout changed the base branch from develop to main May 21, 2023 15:28
@fernanduandrade
Copy link
Contributor Author

Salve @Novout, pode deixar consigo sim

@fernanduandrade fernanduandrade marked this pull request as ready for review June 5, 2023 12:53
@fernanduandrade
Copy link
Contributor Author

fernanduandrade commented Jun 6, 2023

Salve @Novout @kjkGustavo resolvi os conflitos, a principio a lógica tá feita, mas tem muito a ser refinado ainda

  • Adicionei um cron job para fazer o evento ficar disponível baseado na data de inicio e disparar uma mensagem no canal
  • Validação para quando não tiver um evento ativo e ser usado o comando
  • Validação para quando um usuário completar o evento

Exemplo do fluxo no chat:
Screenshot from 2023-06-05 21-40-30

Exemplo do fluxo na DM:
Screenshot from 2023-06-05 21-41-12

Dúvida, não há mais a branch de development?

src/commands/quiz_event.ts Outdated Show resolved Hide resolved
src/http/firebase.ts Outdated Show resolved Hide resolved
src/http/firebase.ts Outdated Show resolved Hide resolved
@Novout
Copy link
Member

Novout commented Jun 16, 2023

@fernanduandrade a branch develop foi removida, mas se quiser eu crio aqui sem problemas.

@fernanduandrade
Copy link
Contributor Author

@Novout boa noite, modificações realizadas 👍

@Novout
Copy link
Member

Novout commented Aug 1, 2023

@fernanduandrade está funcional ao ponto de ser aprovado?

@fernanduandrade
Copy link
Contributor Author

@fernanduandrade está funcional ao ponto de ser aprovado?

boa noite, o fluxo do quiz sim, entretanto o fluxo de dar o exp e coins do usuário não. Essa parte n compreendi muito bem vendo as chamadas da api.

@Novout Novout changed the base branch from main to feat/quiz August 8, 2023 20:20
src/commands/quiz_event.ts Outdated Show resolved Hide resolved
src/commands/quiz_event.ts Outdated Show resolved Hide resolved
src/events/cron/quiz.ts Outdated Show resolved Hide resolved
src/events/cron/quiz.ts Show resolved Hide resolved
src/http/firebase.ts Show resolved Hide resolved
src/http/firebase.ts Outdated Show resolved Hide resolved
src/types.ts Outdated Show resolved Hide resolved
src/commands/index.ts Outdated Show resolved Hide resolved
src/commands/index.ts Outdated Show resolved Hide resolved
src/commands/quiz_event.ts Outdated Show resolved Hide resolved
src/events/discord/channel.ts Outdated Show resolved Hide resolved
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.

4 participants