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

feat(service_version): Add support for environments. #542

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

simonwistow
Copy link
Member

Also Activating and Deactivating an Service Version to and from a particular Environment.

It's possible that we don't want the StageVerson and UnstageVersion shortcuts.

@Integralist
Copy link
Collaborator

👋🏻

I don't see this API behaviour documented:
https://www.fastly.com/documentation/reference/api/services/version/#activate-service-version

What does the additional path segment do with regards to service activation?

@kpfleming
Copy link
Contributor

The API docs are still in progress.

@kpfleming kpfleming changed the title Add suppport for Staging and Unstaging a Service Version feat(service_version): Add support for environments, and 'stage'/'unstage' actions. Aug 28, 2024
Copy link
Contributor

@kpfleming kpfleming left a comment

Choose a reason for hiding this comment

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

This looks fine to me other than properly capitalizing Fastly :-)

We'll wait to merge this until the API documentation has been merged, partially to ensure that the terminology ("Environment") matches up.

fastly/service_version.go Outdated Show resolved Hide resolved
fastly/service_version.go Outdated Show resolved Hide resolved
@kpfleming
Copy link
Contributor

I believe this is ready but asked for a second opinion :-)

…ice-versions.

Add 'environments' details to service and service-version API responses.
@kpfleming kpfleming changed the title feat(service_version): Add support for environments, and 'stage'/'unstage' actions. feat(service_version): Add support for environments. Sep 9, 2024
@kpfleming
Copy link
Contributor

This has been simplified to remove the 'stage' and 'unstage' wrappers. It now only adds the optional 'Environment' parameter to 'Activate'/'Deactivate', and exposes the 'Environments' details included in API responses when service/version details are obtained.

@kpfleming kpfleming merged commit 295e591 into main Sep 9, 2024
4 checks passed
@kpfleming kpfleming deleted the simon/staging branch September 9, 2024 14:17
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