Skip to content

Updated express example, now working. #143

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

Merged
merged 1 commit into from
Jul 29, 2021
Merged

Conversation

okbel
Copy link
Contributor

@okbel okbel commented Jul 29, 2021

There are two variables named res. Fails responding back to user because it's using res from db instance. Renamed to result and it's now a working example.

There are two variables named `res`. Fails responding back to user because it's using res from db. Renamed to `result` now working.
@netlify
Copy link

netlify bot commented Jul 29, 2021

✔️ Hey! A preview of the changes is deployed on netlify to take a look at! :)

🔨 Explore the source changes: 582655f

🔍 Inspect the deploy log: https://app.netlify.com/sites/hopeful-clarke-f36d36/deploys/61029f47b9998b0007823712

😎 Browse the preview: https://deploy-preview-143--hopeful-clarke-f36d36.netlify.app

Copy link
Collaborator

@charmander charmander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@charmander charmander merged commit bad1fe3 into brianc:master Jul 29, 2021
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