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 raw_data_source FK to table #595

Merged
merged 2 commits into from
May 13, 2024
Merged

Conversation

rdahis
Copy link
Member

@rdahis rdahis commented May 10, 2024

Closes #594.

@rdahis rdahis self-assigned this May 10, 2024
@rdahis
Copy link
Member Author

rdahis commented May 10, 2024

Não temos branch de development e staging para casar com o front-end? @vncsna

@vncsna vncsna changed the title add raw_data_source FK to table feat: add raw_data_source FK to table May 10, 2024
@vncsna
Copy link
Contributor

vncsna commented May 10, 2024

@rdahis não temos, o deploy de production e staging é via workflow_dispatch, que é um botão escrito Run workflow na página da action. E o de dev é via adicionar a label preview nesse pull request. Nesse caso o lint python falhou porque a linha tem tamanho maior que 100 caracteres. Alias, está faltando uma migração nesse PR, executar make run_local iniciar a criação dela

image

image

@vncsna vncsna added the preview Preview development deployment label May 10, 2024
@rdahis
Copy link
Member Author

rdahis commented May 10, 2024

@vncsna vê agora?

@vncsna vncsna added preview Preview development deployment and removed preview Preview development deployment labels May 10, 2024
Copy link
Contributor

@vncsna vncsna left a comment

Choose a reason for hiding this comment

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

O admin ficou meio lento ao acessar uma tabela, não sei se vai ser assim em prod

@laura-l-amaral
Copy link
Contributor

mas @rdahis não é pra ser many to many é? uma tabela só pode ter uma unica fonte, apesar de uma fonte poder ter varias tabelas conectadas a ela

@rdahis
Copy link
Member Author

rdahis commented May 10, 2024 via email

@vncsna vncsna merged commit 5652fc0 into main May 13, 2024
5 checks passed
@vncsna vncsna deleted the link_table_raw_data_source branch May 13, 2024 02:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preview Preview development deployment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Conectar fontes originais a tabelas
3 participants