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: replace downloadurl_placeholder with download_link #162

Merged
merged 5 commits into from
Feb 12, 2025

Conversation

chuckadams
Copy link
Contributor

@chuckadams chuckadams commented Feb 12, 2025

Pull Request

What changed?

  • Replaced the placeholder in theme update responses with download_link.

  • updates tests to ensure download_link was used.

  • Various fixes and updates to bruno tests that had to be done alone the way

Why did it change?

obvious bugfix is obvious

Did you fix any specific issues?

none.

CERTIFICATION

By opening this pull request, I do agree to abide by the Code of Conduct and be bound by the terms of the Contribution Guidelines in effect on the date and time of my contribution as proven by the revision information in GitHub.

@chuckadams chuckadams merged commit 9b3f07b into main Feb 12, 2025
1 check passed
@chuckadams chuckadams deleted the chuck/fix/update-packages branch February 12, 2025 01:36
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.

1 participant