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.

Sign up for GitHub

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

Jump to bottom

make z, ra and dec mandatory in back-end #346

Closed
Tracked by #344
gschwend opened this issue Dec 17, 2024 · 1 comment · Fixed by #384 or #393
Closed
Tracked by #344

make z, ra and dec mandatory in back-end #346

gschwend opened this issue Dec 17, 2024 · 1 comment · Fixed by #384 or #393
Assignees
@crisingulani

Comments

@gschwend
Copy link
Member

No description provided.

@gschwend gschwend mentioned this issue Dec 17, 2024
Closed
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Software Dec 17, 2024
@gschwend gschwend changed the title make ra and dec mandatory in back-end make z, ra and dec mandatory in back-end Dec 17, 2024
@gschwend gschwend moved this from 📋 Backlog to 🔖 To do in Software Jan 9, 2025
@crisingulani crisingulani moved this from 🔖 To do to 🏗 In progress in Software Jan 24, 2025
@crisingulani crisingulani linked a pull request Jan 28, 2025 that will close this issue
Mandatory cols #384
Merged
@crisingulani crisingulani moved this from 🏗 In progress to 👀 In review in Software Jan 29, 2025
@gschwend
Copy link
Member Author

gschwend commented Feb 4, 2025

@crisingulani eu esqueci de especificar aqui (coloquei só lá no issue do front-end) que a coluna z (redshift) também é obrigatória no caso de data product type ser Training Set (sorry!). Por favor adicione aqui no mesmo PR esta alteração.

@gschwend gschwend reopened this Feb 6, 2025
@gschwend gschwend moved this from 👀 In review to 🏗 In progress in Software Feb 6, 2025
@crisingulani crisingulani moved this from 🏗 In progress to 👀 In review in Software Feb 7, 2025
@crisingulani crisingulani linked a pull request Feb 11, 2025 that will close this issue
Maked z mandatory in training set type. #393
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@crisingulani crisingulani

Labels
None yet
Projects
Software
Status: 👀 In review
Milestone
No milestone
2 participants
@crisingulani @gschwend