Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

Allow site key to be configured by environment #1573

Merged
merged 4 commits into from
Oct 28, 2020

Conversation

oliverpool
Copy link
Contributor

@oliverpool oliverpool commented Sep 28, 2020

Closes #1567

@oliverpool
Copy link
Contributor Author

@ashleymichal do you have any feedback on this PR ? (or an expected delay until it gets reviewed?)

@nataliescottdavidson
Copy link
Contributor

@oliverpool update and rebase and I will approve.

@nataliescottdavidson nataliescottdavidson merged commit bf0601a into cloudflare:master Oct 28, 2020
@nataliescottdavidson nataliescottdavidson added this to the 1.12.0 milestone Oct 28, 2020
oliverpool added a commit to oliverpool/cloudflare-docs that referenced this pull request Oct 28, 2020
adamschwartz pushed a commit to cloudflare/cloudflare-docs that referenced this pull request Oct 28, 2020
nataliescottdavidson added a commit that referenced this pull request Oct 30, 2020
This reverts commit bf0601a.

Change caused a regression which destroyed deployments.
#1625
nataliescottdavidson added a commit that referenced this pull request Oct 30, 2020
This reverts commit bf0601a.

Change caused a regression which destroyed deployments.
#1625
ObsidianMinor added a commit that referenced this pull request Nov 1, 2020
ObsidianMinor pushed a commit to ObsidianMinor/wrangler that referenced this pull request Nov 2, 2020
…)" (cloudflare#1626)

This reverts commit bf0601a.

Change caused a regression which destroyed deployments.
cloudflare#1625
ObsidianMinor added a commit that referenced this pull request Nov 2, 2020
ObsidianMinor added a commit that referenced this pull request Nov 2, 2020
ObsidianMinor added a commit that referenced this pull request Nov 2, 2020
Revert "Revert "Allow site key to be configured by environment (#1573)""
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Inconsistency for configuration inheritance
2 participants