Skip to content

feat(api): add endpoint to update site aliases#551

Merged
saeedvaziry merged 3 commits intovitodeploy:2.xfrom
J4Wx:2.x
Mar 21, 2025
Merged

feat(api): add endpoint to update site aliases#551
saeedvaziry merged 3 commits intovitodeploy:2.xfrom
J4Wx:2.x

Conversation

@J4Wx
Copy link
Contributor

@J4Wx J4Wx commented Mar 17, 2025

Allows the updating of a sites aliases using a PUT request. The request requires one property on the body, "aliases" - which is an array of strings.

I have not ran scribe:generate (yet) - the entire scribe directory gets rewritten and I'm not familiar enough with scribe to know if this is the correct behaviour.

I intend to continue adding more API functionality, but this is something that has a particular priority for me currently (hence PRing this alone first).

@saeedvaziry
Copy link
Member

Thanks for the PR. I will run the scribe:generate myself. however I would appreciate one test to cover this endpoint.

@J4Wx
Copy link
Contributor Author

J4Wx commented Mar 18, 2025

Test added!

It's possible that I'm just an idiot, but it might be worth adding to the contributing docs (or maybe just the readme?) that you need to manually create storage/database-test.sqlite before running the tests.

It only took 30 seconds for me to realise, but might help some people.

@saeedvaziry saeedvaziry merged commit 03be2d3 into vitodeploy:2.x Mar 21, 2025
3 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.

2 participants