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

Add auth method group rule type to Access group #762

Merged

Conversation

Justin-Holmes
Copy link
Contributor

Related: cloudflare/cloudflare-go#516

Tests will fail until the the cloudflare-go pkg is updated with the above change.

@Justin-Holmes Justin-Holmes changed the title Add auth method group rule type to Access group [WIP] Add auth method group rule type to Access group Aug 13, 2020
@Justin-Holmes Justin-Holmes force-pushed the justin/add-auth-method-rule branch from fdd5342 to a4f3ce1 Compare August 14, 2020 15:39
@jacobbednarz jacobbednarz marked this pull request as draft August 17, 2020 22:11
@jacobbednarz
Copy link
Member

@Justin-Holmes I've converted this to a draft PR while we're waiting the upstream changes to be pulled in.

@jacobbednarz
Copy link
Member

I've pulled in the latest master branch for this one however it looks like it's now getting a struct field error. Would you mind having a dig into it and we can look to get this one merged?

I forgot to update the field name when I copy/pasted
@Justin-Holmes Justin-Holmes changed the title [WIP] Add auth method group rule type to Access group Add auth method group rule type to Access group Aug 25, 2020
@Justin-Holmes Justin-Holmes marked this pull request as ready for review August 25, 2020 13:36
@jacobbednarz jacobbednarz merged commit 9dc578c into cloudflare:master Sep 1, 2020
@Justin-Holmes Justin-Holmes deleted the justin/add-auth-method-rule branch September 1, 2020 20:30
@jacobbednarz
Copy link
Member

This is great, cheers!

boekkooi-lengoo pushed a commit to boekkooi-lengoo/terraform-provider-cloudflare that referenced this pull request Feb 28, 2022
…privateappsTF

adds private app type and new fields
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