Skip to content

Mapeandos os valores de TrackStatus para String #50

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

Conversation

victorlourenco-vhl
Copy link
Collaborator

Nessa task ocorreu o mapeamento dos valores de TrackStatus para String, conforme a imagem abaixo, e a remoção das culunas de encoder direito, esquerdo e a coluna de status.

image

encMedia: Robot.MappingRecord['encMedia'],
encLeft: Robot.MappingRecord['encLeft'],
encRight: Robot.MappingRecord['encRight'],
Copy link
Collaborator

Choose a reason for hiding this comment

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

Por que tirou essas linhas? As funcionalidades da página de mapeamento estão funcionando ainda?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

O Mat pediu pediu para remover as colunas de Status, Encoder Direito e Esquerdo, aí fui removendo os códigos dessas colunas onde a IDE estava apontando alguns erros.

package.json Outdated
@@ -6,7 +6,7 @@
"author": "Raphael Henrique Nunes de Freitas <[email protected]>",
"private": true,
"scripts": {
"dev": "QENV=development quasar dev",
"dev": "set \"QENV=development\" && quasar dev",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Reverte esse commit pls. Melhor deixar no formato do Linux porque já tem o devcontainer no projeto.

v-slot="{ version }"
v-if="showVersionsDialog"
>
<ProfileVersionsDialog collection="mappings" title="Mapeamentos" :data="mappingRecords" @install-request="(version: Robot.ProfileVersion<Robot.Mapping>) =>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Passa o ESLint e o Prettier nas suas mudanças pls? Para deixar o código dentro do padrão do restante do projeto.

@Matheus-de-Sousa
Copy link
Collaborator

As alterações feitas estão funcionando, entretanto tem um pequeno problema no envio do mapeamento, as funções que estão incorretas estão no arquivo src/composables/mapping.ts e são as funções deserializeRecord e serializeRecord basicamente você esqueceu de tirar as colunas do mapeamento que deixaram de ser utilizadas do envio do mapeamento.
Na imagem abaixo o código está corrigido
image

@Matheus-de-Sousa
Copy link
Collaborator

Faça as correções solicitadas por mim e pelo William. Após isso, faça o pull request a partir de uma branch contida neste reoisutório derivada diretamente da develop ao invés de usar o fork, isso permitirá ter um registro mais centralizado das versões do código

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