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

Toggle mocked datastore/integrations based on deployment state #92

Merged
merged 2 commits into from
Aug 20, 2023

Conversation

AddisonTustin
Copy link
Collaborator

No description provided.

@nx-cloud
Copy link

nx-cloud bot commented Aug 19, 2023

☁️ Nx Cloud Report

CI is running/has finished running commands for commit ae481d8. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


✅ Successfully ran 4 targets

Sent with 💌 from NxCloud.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Aug 19, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: ae481d8
Status: ✅  Deploy successful!
Preview URL: https://8dd82de3.polyratings.pages.dev
Branch Preview URL: https://locally-deployed.polyratings.pages.dev

View logs

@AddisonTustin AddisonTustin force-pushed the locally-deployed branch 4 times, most recently from d179472 to 85fd7f0 Compare August 19, 2023 23:59
@AddisonTustin AddisonTustin marked this pull request as ready for review August 20, 2023 00:01
@mfish33 mfish33 self-requested a review August 20, 2023 01:23
@mfish33 mfish33 merged commit f690d20 into local-dev Aug 20, 2023
2 checks passed
mfish33 added a commit that referenced this pull request Aug 20, 2023
* locally develop with CF workers

* fix env types

* make rating analyzer have a new format to increase modularity

* removed console log

* readme updates

* Toggle mocked datastore/integrations based on deployment state (#92)

* Toggle mocked datastore/integrations based on deployment state

* tiny changes

---------

Co-authored-by: mfish33 <maxmfishernj@gmail.com>

* still not working

* fix broken package lock npm/cli#4828

---------

Co-authored-by: Chris Lawson <Chris2fourlaw@users.noreply.github.com>
Co-authored-by: AddisonTustin <addison@atustin.dev>
AddisonTustin added a commit that referenced this pull request Sep 23, 2023
* Add anonymized identifiers to submitted reports (#87)

* chore: update versions (#91)

* chore: update versions

* fix: restore previous click behavior

* Locally Develop with CF Workers (#89)

* locally develop with CF workers

* fix env types

* make rating analyzer have a new format to increase modularity

* removed console log

* readme updates

* Toggle mocked datastore/integrations based on deployment state (#92)

* Toggle mocked datastore/integrations based on deployment state

* tiny changes

---------

Co-authored-by: mfish33 <maxmfishernj@gmail.com>

* still not working

* fix broken package lock npm/cli#4828

---------

Co-authored-by: Chris Lawson <Chris2fourlaw@users.noreply.github.com>
Co-authored-by: AddisonTustin <addison@atustin.dev>

* Rolling ids (#95)

* rolling ids

* add long about comment

* fix missing namespace

* fix lint

* Rating Anonymous Identifiers (#94)

* rating anonymous identifiers

* fix lint and properly parse perspective response

* linting fix

* make all professor key a shared constant

* Use anonymousIdDao for rating id

---------

Co-authored-by: Addison Tustin <addison@atustin.dev>

* Stop preview deployments from trying to target localhost (#96)

* Stop preview deployments from defaulting to localhost

* change local to local-dev to avoid .env collisions

* fix single quote

* select right configuration at build time (#97)

---------

Co-authored-by: AddisonTustin <addison@atustin.dev>
Co-authored-by: Chris Lawson <Chris2fourlaw@users.noreply.github.com>
@AddisonTustin AddisonTustin deleted the locally-deployed branch September 23, 2023 22:20
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