Skip to content

Commit

Permalink
Merge pull request #121 from RedisLabs/add-missing-json-conversion-rules
Browse files Browse the repository at this point in the history
Added some missing json conversion rules and deleted some dud comments
  • Loading branch information
JohnSharpe committed Jul 26, 2023
2 parents 4ddc60c + 9cc3336 commit 852fe5b
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions service/access_control_lists/roles/model.go
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ import "github.com/RedisLabs/rediscloud-go-api/internal"
type ListRolesResponse struct {
AccountId *int `json:"accountId,omitempty"`
Roles []*GetRoleResponse `json:"roles,omitempty"`
// Links []*AnotherThing
}

type GetRoleResponse struct {
Expand All @@ -23,9 +22,9 @@ func (o GetRoleResponse) String() string {
}

type GetRuleInRoleResponse struct {
RuleId *int
RuleName *string
Databases []*GetDatabaseInRuleInRoleResponse
RuleId *int `json:"ruleId,omitempty"`
RuleName *string `json:"ruleName,omitempty"`
Databases []*GetDatabaseInRuleInRoleResponse `json:"databases,omitempty"`
}

func (o GetRuleInRoleResponse) String() string {
Expand All @@ -36,7 +35,7 @@ type GetDatabaseInRuleInRoleResponse struct {
SubscriptionId *int `json:"subscriptionId,omitempty"`
DatabaseId *int `json:"databaseId,omitempty"`
DatabaseName *string `json:"databaseName,omitempty"`
Regions []*string `json:"regions,omitempty"` // Docs are unclear
Regions []*string `json:"regions,omitempty"`
}

func (o GetDatabaseInRuleInRoleResponse) String() string {
Expand Down Expand Up @@ -71,7 +70,7 @@ func (o CreateRuleInRoleRequest) String() string {
type CreateDatabaseInRuleInRoleRequest struct {
SubscriptionId *int `json:"subscriptionId,omitempty"`
DatabaseId *int `json:"databaseId,omitempty"`
Regions []*string `json:"regions,omitempty"` // Docs are unclear
Regions []*string `json:"regions,omitempty"`
}

func (o CreateDatabaseInRuleInRoleRequest) String() string {
Expand Down

0 comments on commit 852fe5b

Please sign in to comment.