Skip to content
This repository has been archived by the owner on May 21, 2022. It is now read-only.

Add new Datapackage Descriptor #170

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add new Datapackage Descriptor #170

wants to merge 2 commits into from

Conversation

joeberth
Copy link
Contributor

Copy link
Contributor

@danielfireman danielfireman left a comment

Choose a reason for hiding this comment

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

Bacana!

Além dos comentários que fiz abaixo, também está faltando alterar a versão dos dados.

packager/datapackage_descriptor.json Show resolved Hide resolved
packager/datapackage_descriptor.json Outdated Show resolved Hide resolved
packager/datapackage_descriptor.json Outdated Show resolved Hide resolved
packager/datapackage_descriptor.json Outdated Show resolved Hide resolved
packager/datapackage_descriptor.json Outdated Show resolved Hide resolved
"type": "number",
"description": "Represents the sum of discounts applied to the employee.",
"description-ptbr": "Total de descontos aplicados ao empregado",
"name": "nome_funcionario",
Copy link
Contributor

Choose a reason for hiding this comment

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

Não precisamos do nome do empregado nesse recurso. Ele pode ser facilmente obtido cruzando com o outro recurso via id_funcionario

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Meu ponto de colocar aqui foi pensando no cara que quer ver o que foi pago detalhado e vai ver apenas o ID do cara, mas teria que abrir outra planilha pra achar o nome, mas OK, retirar.

Copy link
Contributor

Choose a reason for hiding this comment

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

Beleza. Lembrando que o foco desse layout é facilitar a análise, acho que isso vai fazer o arquivo crescer demais esse arquivo, para uma operação que é super simples de fazer se você estiver usando ferramentas de análises. Para debug e outras coisas, iremos direto no banco de dados.

Dito isso, poderia remover, por favor?

packager/datapackage_descriptor.json Outdated Show resolved Hide resolved
packager/datapackage_descriptor.json Outdated Show resolved Hide resolved
packager/datapackage_descriptor.json Outdated Show resolved Hide resolved
packager/datapackage_descriptor.json Outdated Show resolved Hide resolved
Copy link
Contributor

@danielfireman danielfireman left a comment

Choose a reason for hiding this comment

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

Caminho está bacana demais! Fiz mais alguns comentários abaixo. Por favor endereçar antes de fazer merge.

"constraints": {
"minimum": "0"
}
"type": "boolean",
Copy link
Contributor

Choose a reason for hiding this comment

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

Acho que esse campo deve ser obrigatório.

"type": "number",
"description": "Represents the sum of discounts applied to the employee.",
"description-ptbr": "Total de descontos aplicados ao empregado",
"name": "nome_funcionario",
Copy link
Contributor

Choose a reason for hiding this comment

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

Beleza. Lembrando que o foco desse layout é facilitar a análise, acho que isso vai fazer o arquivo crescer demais esse arquivo, para uma operação que é super simples de fazer se você estiver usando ferramentas de análises. Para debug e outras coisas, iremos direto no banco de dados.

Dito isso, poderia remover, por favor?

@@ -334,7 +336,7 @@
],
"name": "remuneracoes-jusbr",
"title": "Remunerações do Sistema de Justiça Brasileiro",
"description": "Remunerações do Sistema de Justiça Brasileiro, libertadas por DadosJusBR",
"description-en": "Remunerações do Sistema de Justiça Brasileiro, libertadas por DadosJusBR",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"description-en": "Remunerações do Sistema de Justiça Brasileiro, libertadas por DadosJusBR",
"description": "Remunerações do Sistema de Justiça Brasileiro, libertadas por dadosjusbr.org",
"description-en": "Brazilian Justice System remunerations, released by dadosjusbr.org",

@@ -334,7 +336,7 @@
],
"name": "remuneracoes-jusbr",
"title": "Remunerações do Sistema de Justiça Brasileiro",
"description": "Remunerações do Sistema de Justiça Brasileiro, libertadas por DadosJusBR",
"description-en": "Remunerações do Sistema de Justiça Brasileiro, libertadas por DadosJusBR",
"homepage": "https://dadosjusbr.org",
"version": "1.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"version": "1.0.0",
"version": "2.0.0",

@nazareno
Copy link
Contributor

nazareno commented Jul 1, 2021

@danielfireman , esse PR deveria ter ido para o main?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants