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

Set room text field to lower case by default #160

Merged
merged 2 commits into from
Aug 29, 2017

Conversation

idelgado
Copy link
Contributor

@idelgado idelgado commented Aug 25, 2017

When developers start by creating a token in the Twilio console and specify a room name when generating a token the default capitalization on the page is lower case while in this app it's upper case which can lead to confusion when getting started since room names are case sensitive

Copy link
Contributor

@ceaglest ceaglest left a comment

Choose a reason for hiding this comment

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

This is very cool! Does the CallKit QS have the same issue? If so would love to make the change in that app as well.

@idelgado
Copy link
Contributor Author

Good point. I'll check the examples and the obj-c quickstart as well.

Copy link
Contributor

@ceaglest ceaglest left a comment

Choose a reason for hiding this comment

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

+1. Please squash when merging.

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