Skip to content

Módulo Memória#27

Open
lvolcov wants to merge 4 commits into
mainfrom
feat/memoria
Open

Módulo Memória#27
lvolcov wants to merge 4 commits into
mainfrom
feat/memoria

Conversation

@lvolcov
Copy link
Copy Markdown
Contributor

@lvolcov lvolcov commented Jan 10, 2021

#12

@jampow jampow added the enhancement New feature or request label Jan 13, 2021
@jampow jampow added this to the Versão 1.0.0 milestone Jan 13, 2021
@lvolcov
Copy link
Copy Markdown
Contributor Author

lvolcov commented Jan 24, 2021

Percebi só agora que tem respostas que pedem pra apertar o mesmo número apertado em etapas anteriores e outras que pedem pra apertar a mesma posição apertada em etapas anteriores, eu tava fazendo todos esses casos retornando o mesmo número apertado, vou subir um commit arrumando isso e arrumando as descrições dos testes.

jampow
jampow previously approved these changes Feb 8, 2021
Comment thread src/622/modules/memoria/index.js Outdated
@@ -0,0 +1,58 @@
/**
* Função que resolve o módulo Memória
* @param {string[]} previousStep - Array que traz as arrays das interações anteriores e suas respostas
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Não entendi, o tipo é um Array de strings, mas na descrição vc diz q é um Array de arrays... 🤔

Comment thread src/622/modules/memoria/index.js Outdated
throw 'CURRENT_STEP_INCOMPLETED'
}
let answer
switch(previousSteps.length) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

trocar switchcases aninhados por objeto como no módulo genius

@jampow jampow changed the base branch from master to main April 26, 2022 04:37
@jampow jampow assigned jampow and unassigned lvolcov Jun 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants