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

chore: rm LINKDEX_URL from wrangler.toml #2023

Merged
merged 2 commits into from Oct 27, 2022
Merged

chore: rm LINKDEX_URL from wrangler.toml #2023

merged 2 commits into from Oct 27, 2022

Conversation

olizilla
Copy link
Contributor

We're now using the generated lambda Function URL directly and setting it in the ENV via the dashboard / wranger cli.

I've already updated the LINKDEX_URL var for staging and prod. This change makes it so subsequent deployments don't overwrite that change.

If you change your environment variables in the Cloudflare dashboard, Wrangler will override them the next time you deploy.
– https://developers.cloudflare.com/workers/wrangler/configuration/#source-of-truth

see: web3-storage/linkdex-api#7

License: MIT
Signed-off-by: Oli Evans oli@protocol.ai

We're now using the generated lambda Function URL directly and setting it in the ENV via the dashboard / wranger cli.

see: web3-storage/linkdex-api#7

License: MIT
Signed-off-by: Oli Evans <oli@protocol.ai>
@olizilla olizilla temporarily deployed to production October 12, 2022 13:40 Inactive
@olizilla olizilla temporarily deployed to production October 27, 2022 09:39 Inactive
@olizilla olizilla merged commit 1341283 into main Oct 27, 2022
@olizilla olizilla deleted the rm-linkdex-url branch October 27, 2022 09:40
olizilla pushed a commit that referenced this pull request Oct 27, 2022
🤖 I have created a release *beep* *boop*
---


##
[7.11.0](api-v7.10.0...api-v7.11.0)
(2022-10-27)


### Features

* add metadata to upload error reports
([#2070](#2070))
([6d240b7](6d240b7))


### Other Changes

* rm LINKDEX_URL from wrangler.toml
([#2023](#2023))
([1341283](1341283))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

None yet

2 participants