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

Fix: tree details broonie #798

Merged
merged 5 commits into from
Jan 21, 2025
Merged

Fix: tree details broonie #798

merged 5 commits into from
Jan 21, 2025

Conversation

Francisco2002
Copy link
Collaborator

A tree in broonie origin have no git url/branch (null field).
In this PR, I remove de validation on frontend to allow the user send a empty git url and git branch to the tree details endpoint, that now returns a error if no data was found.
In the backend, add a logic to search this trees when this fields are null.

How to test

  • go to any tree and check if the behavior is the same of the staging
  • go to broonie origin and search for mainline master tree

Close #729

@Francisco2002 Francisco2002 self-assigned this Jan 21, 2025
@Francisco2002 Francisco2002 force-pushed the fix/tree-details-broonie branch from 370ddd0 to 02ca16a Compare January 21, 2025 16:37
@WilsonNet
Copy link
Collaborator

graph is returning the wrong data
image

Copy link
Collaborator

@WilsonNet WilsonNet left a comment

Choose a reason for hiding this comment

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

please create a ticket to the graph before merging

Copy link
Collaborator Author

@Francisco2002 Francisco2002 left a comment

Choose a reason for hiding this comment

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

Ticket created: #803

@Francisco2002 Francisco2002 force-pushed the fix/tree-details-broonie branch from 02ca16a to 2f7bcf6 Compare January 21, 2025 18:19
Copy link
Collaborator

@MarceloRobert MarceloRobert left a comment

Choose a reason for hiding this comment

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

Seems to be working, this also fixes a lot if not all tuxsuite trees too

@Francisco2002 Francisco2002 force-pushed the fix/tree-details-broonie branch from 2f7bcf6 to bbc9b75 Compare January 21, 2025 18:59
@Francisco2002 Francisco2002 merged commit c740c19 into main Jan 21, 2025
5 checks passed
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.

Tree Details rendering with error in broonie origin
3 participants