-
Notifications
You must be signed in to change notification settings - Fork 178
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
INTMDB-144: Updated for scopes database users #368
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A quick check on a path change and a question (non blocking but may result in a 2nd ticket). Thanks!
go.mod
Outdated
@@ -1,7 +1,7 @@ | |||
module github.com/mongodb/terraform-provider-mongodbatlas | |||
|
|||
go 1.14 | |||
|
|||
replace go.mongodb.org/atlas => ../go-client-mongodb-atlas |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably shouldn't leave, looks like for testing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, so I can use the recent changes of client mongodb, since it's merged I will update from master mongodb client
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Got ticket in for issue I saw too and noted that. Thanks!
@marinsalinas or @leofigy can one of you review please? |
v := val.(map[string]interface{}) | ||
res[i] = matlas.Scope{ | ||
scope := matlas.Scope{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks better
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
scopes
and send an array empty if not specifiedTestAccResourceMongoDBAtlasDatabaseUser_withScopesAndEmpty
Link to any related issue(s):
Type of change:
Required Checklist:
Further comments
Waiting for PR to be merged then update the vendor from master and merge