Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add support for Webtop management
This pull request implements the feature request outlined in issue #107, adding support for Webtop Creation, Deletion, Modification, and Retrieval to the F5 BIG-IP Go SDK.
Changes Implemented
apm.go
:WebtopType
,CustomizationType
,InitialState
, andLinkType
BooledString
for handling boolean values represented as strings in JSON payloadsWebtop
,WebtopConfig
, andWebtopRead
structs inapm.go
.BigIP
struct inapm.go
:CreateWebtop(ctx context.Context, webtop Webtop) error
DeleteWebtop(ctx context.Context, name string) error
GetWebtop(ctx context.Context, name string) (*WebtopRead, error)
ModifyWebtop(ctx context.Context, name string, webtop WebtopConfig) error
cmd/webtop/example.go
to demonstrate the usage of the new Webtop functionality.Implementation Details
Webtop
struct represents the configuration for creating or updating a Webtop.WebtopRead
struct includes additional fields returned when retrieving a Webtop.BooledString
type handles the conversion between boolean values and their string representations in the API.Testing
cmd/webtop/example.go
.Additional Notes
ModifyWebtop
method only allows updating theWebtopConfig
fields, as per the API specifications.Related Issue
Closes #107
Please review and let me know if any further changes or information are needed.