Adiciona exsicata_tipo - #156
vitorRibeiro7 wants to merge 12 commits into
Conversation
…iação e edição do tombo
|
|
||
| export async function run(knex: Knex): Promise<void> { | ||
| await knex.transaction(async trx => { | ||
| await trx.raw(` |
There was a problem hiding this comment.
A mudança na estrutura não precisa estar dentro de uma transação, não faz diferença.
|
|
||
| export async function run(knex: Knex): Promise<void> { | ||
| await knex.transaction(async trx => { | ||
| await trx.raw(` |
There was a problem hiding this comment.
Utiliza o esquema de migração já existe no Knex, por favor. Vai facilitar quando a gente for mudar de MySQL para Postgres.
| ADD COLUMN exsicata_tipo ENUM('UNICATA', 'DUPLICATA') NULL; | ||
| `) | ||
|
|
||
| const rows = parseFile( |
There was a problem hiding this comment.
Duas coisas aqui.
- Acho que não me fiz entender no dia. Cria o diretório com o .csv dentro do diretório
migratione não fora. - Quando for ler o arquivo, utiliza o caminho relativo ao arquivo atual utilizando o
pathnode Node, por exemplopath.join(__dirname, 'caminho-arquivo.csv')
| for await (const row of rows) { | ||
| const { id, tombo_tipo: tomboTipo } = row | ||
|
|
||
| if (!tomboTipo) return |
There was a problem hiding this comment.
Se um tombo não tem valor em tombo_tipo, ele deveria estar no CSV?
| await knex.transaction(async trx => { | ||
| await trx.raw(` | ||
| ALTER TABLE tombos | ||
| ADD COLUMN exsicata_tipo ENUM('UNICATA', 'DUPLICATA') NULL; |
There was a problem hiding this comment.
Por que mudamos de true/false para um ENUM? Embora hoje possa existir o null, ele deveria ser um valor lógico sim ou não. Criar um ENUM não me parece fazer sentido nesse caso, pois não teremos mais de dois valores no futuro.
| options: [{ min: 3 }], | ||
| }, | ||
| }, | ||
| 'json.exsicataTipo': { |
There was a problem hiding this comment.
Utilize underline para manter o padrão do projeto.
| options: [{ min: 3 }], | ||
| }, | ||
| }, | ||
| 'json.exsicataTipo': { |
There was a problem hiding this comment.
Utilize underline para manter o padrão do projeto.
…plicata-na-tabela-tombo
…plicata-na-tabela-tombo
…ta-ou-duplicata-na-tabela-tombo" This reverts commit 50f5da5.
…plicata-na-tabela-tombo
…plicata-na-tabela-tombo
2 participants
Adicionado a criação do campo exsicata_tipo na tabela tombos, inserido o fluxo dentro da migration.
Processa um csv para os tombos do firebird que contem as substrings "unicata" ou "duplicata" no campo [obs_tombo].
Inclui também o novo campo nas rotas de criação e edição do tombo.