Skip to content
This repository was archived by the owner on May 22, 2023. It is now read-only.

Code Exchange Upgrade#7

Merged
po5i merged 4 commits into
nextfrom
code-exchange-upgrade
May 13, 2020
Merged

Code Exchange Upgrade#7
po5i merged 4 commits into
nextfrom
code-exchange-upgrade

Conversation

@po5i

@po5i po5i commented Apr 20, 2020

Copy link
Copy Markdown
Contributor

Resolves #6

@po5i po5i force-pushed the code-exchange-upgrade branch from 32764d8 to 3240c16 Compare April 20, 2020 22:28
@po5i po5i force-pushed the code-exchange-upgrade branch from 3240c16 to 8f5646e Compare April 20, 2020 22:31
@po5i po5i force-pushed the code-exchange-upgrade branch from 7799800 to 5a1ba45 Compare April 21, 2020 20:30
Comment thread README.md
Comment thread README.md
Comment thread README.md
@@ -103,6 +99,9 @@ We'd recommend Google Chrome or Mozilla Firefox instead.

![screenshot of phone app](https://s3.amazonaws.com/com.twilio.prod.twilio-docs/images/TwilioClientQuickstart.original.png)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This app doesnt seem to work for me. I constantly get Twilio.Device Error: Invalid Access Token issuer/subject

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Are you testing locally with the code-exchange-upgrade branch?
Are TWILIO_ACCOUNT_SID, API_KEY, and API_SECRET values correct in your .env file?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

not sure what changed, but this started working for me again. are we correctly setting the environment variables whenever the app is run? or do we expect the user to manually source .env after making changes to the file?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's not needed to source .env. This change was introduced in this PR.

@po5i po5i changed the base branch from master to next May 11, 2020 15:29
@po5i po5i merged commit 8806350 into next May 13, 2020
@po5i po5i deleted the code-exchange-upgrade branch May 13, 2020 13:20
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.

Code Exchange quality checklist

2 participants