Skip to content

TA-4668: Update documentation for get one version metadata command#291

Merged
Zgjim Haziri (ZgjimHaziri) merged 3 commits into
masterfrom
TA-4668-update-documentation-of-get-one-version-command
Dec 15, 2025
Merged

TA-4668: Update documentation for get one version metadata command#291
Zgjim Haziri (ZgjimHaziri) merged 3 commits into
masterfrom
TA-4668-update-documentation-of-get-one-version-command

Conversation

@Buqeta
Copy link
Copy Markdown
Contributor

Description

After adding the LATEST keyword to the version path variable of the find one version endpoint, created this PR to update the documentation of the CLI command that uses that endpoint.

Relevant links

Checklist

  • I have self-reviewed this PR
  • I have tested the change and proved that it works in different scenarios
  • I have updated docs if needed

@Buqeta Buqeta (Buqeta) requested a review from a team as a code owner December 3, 2025 14:05
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we also update the Documentation.md file?

@Buqeta
Copy link
Copy Markdown
Contributor Author

Should we also update the Documentation.md file?

Updated it.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Dec 4, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Copy link
Copy Markdown
Contributor

@jetakasabaqi Jeta (jetakasabaqi) left a comment

Choose a reason for hiding this comment

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

I would merge this only after the API change is released.

@ZgjimHaziri
Copy link
Copy Markdown
Contributor

I would merge this only after the API change is released.

We can maybe just line-up the actual Content CLI release creation to be after the one in Pacman, if we don't want a long living PR.

@Buqeta
Copy link
Copy Markdown
Contributor Author

I would merge this only after the API change is released.

We can maybe just line-up the actual Content CLI release creation to be after the one in Pacman, if we don't want a long living PR.

Sure, we can do that too. I'll remind the next release manager. I think it's you next week. 😄

@jetakasabaqi
Copy link
Copy Markdown
Contributor

I would merge this only after the API change is released.

We can maybe just line-up the actual Content CLI release creation to be after the one in Pacman, if we don't want a long living PR.

Sure, we can do that too. I'll remind the next release manager. I think it's you next week. 😄

The documentation changes will be live immediately, so that won't work in this case.

@Buqeta
Copy link
Copy Markdown
Contributor Author

I would merge this only after the API change is released.

We can maybe just line-up the actual Content CLI release creation to be after the one in Pacman, if we don't want a long living PR.

Sure, we can do that too. I'll remind the next release manager. I think it's you next week. 😄

The documentation changes will be live immediately, so that won't work in this case.

Oh right, I changed the .md file too. We can leave this PR open until then. It's fine by me.

@ZgjimHaziri Zgjim Haziri (ZgjimHaziri) merged commit 651469d into master Dec 15, 2025
4 of 5 checks passed
@ZgjimHaziri Zgjim Haziri (ZgjimHaziri) deleted the TA-4668-update-documentation-of-get-one-version-command branch December 15, 2025 09:37
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.

6 participants