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

v1: include compose_id in clone response #870

Merged
merged 1 commit into from
Dec 8, 2023

Conversation

ezr-ondrej
Copy link
Collaborator

@ezr-ondrej ezr-ondrej commented Oct 17, 2023

Clones should include their parent ID, to easily match them.
This follows rest standards.

I've got no idea how to do this properly, but I just felt it would be really nice to have compose_id in the clone detail response 🤔

@app-sre-bot
Copy link
Collaborator

Can one of the admins verify this patch?

Copy link

This PR is stale because it has been open 30 days with no activity. Remove "Stale" label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the Stale label Nov 17, 2023
@ezr-ondrej ezr-ondrej changed the title return compose_id for clone api: include compose_id in clone response Nov 20, 2023
@ezr-ondrej ezr-ondrej force-pushed the return_compose_id branch 4 times, most recently from 81d605a to 8d7e48e Compare November 20, 2023 11:48
@ezr-ondrej ezr-ondrej changed the title api: include compose_id in clone response v1: include compose_id in clone response Nov 20, 2023
croissanne
croissanne previously approved these changes Nov 20, 2023
Copy link
Member

@croissanne croissanne left a comment

Choose a reason for hiding this comment

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

lgtm

@ezr-ondrej ezr-ondrej force-pushed the return_compose_id branch 2 times, most recently from 380a116 to 14ce777 Compare November 20, 2023 11:58
@ezr-ondrej ezr-ondrej closed this Nov 20, 2023
@ezr-ondrej ezr-ondrej reopened this Nov 20, 2023
@ezr-ondrej ezr-ondrej force-pushed the return_compose_id branch 2 times, most recently from 00e9d5a to c76df9d Compare November 20, 2023 13:38
croissanne
croissanne previously approved these changes Nov 20, 2023
Copy link
Member

@croissanne croissanne left a comment

Choose a reason for hiding this comment

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

thanks!

@ezr-ondrej
Copy link
Collaborator Author

🍏 🎉

@github-actions github-actions bot removed the Stale label Nov 21, 2023
@croissanne
Copy link
Member

@ezr-ondrej ugh I didn't press the button ._. can you rebase, happy to ack again

Clones should include their parent ID, to easily match them.
It follows the rest guidelines to be traversable.
@ezr-ondrej
Copy link
Collaborator Author

Sure :) here it goes! :)

@croissanne
Copy link
Member

/retest

@ezr-ondrej
Copy link
Collaborator Author

🍏 🎉

@croissanne croissanne enabled auto-merge (rebase) December 8, 2023 14:34
@croissanne croissanne merged commit 7c929f3 into osbuild:main Dec 8, 2023
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.

3 participants