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

docs(deep-links): clarify apple-app-site-association file #2963

Merged
merged 2 commits into from
May 22, 2020

Conversation

jaydrogers
Copy link
Contributor

I was working with the apple-app-site-association file today, banging my head against the wall why it wasn't working.

Then I realized there was a comment at the top of the file that was causing my file to be interpreted as a file to be downloaded (not a content-type of JSON). LOL, programming...

I created this PR to hopefully help clarify the example a little bit more and prevent others from running into the same issue (although I know I am the dumb one 😀)

Thanks for your hard work 🙏

Let me know if you have any questions!

@jaydrogers jaydrogers changed the title Clarifiy the example of the apple-app-site-association Clarifiy the example of the apple-app-site-association file May 21, 2020
@jaydrogers jaydrogers changed the title Clarifiy the example of the apple-app-site-association file Docs: Clarifiy the example of the apple-app-site-association file May 21, 2020
@jaydrogers jaydrogers changed the title Docs: Clarifiy the example of the apple-app-site-association file docs(deep-links): clarify apple-app-site-association file May 21, 2020
@dotNetkow
Copy link
Contributor

Oh, nooooooo sorry Jay! I intended those comments to be illustrative/helpful. Had no idea that would happen, but makes sense. The main issue is that Apple should accept a file extension (.json) - then this wouldn't happen....

Well, thank you so much for opening a PR. Really appreciate it!

@jaydrogers
Copy link
Contributor Author

Hahaha, no problem. I 100% agree about Apple just needs to accept .json. I guess it's just a part of their campaign to "think different" 🤷‍♂️

Thanks again for all of your hard work. You guys are building amazing things!

@jcesarmobile jcesarmobile changed the title docs(deep-links): clarify apple-app-site-association file docs(deep-links): clarify apple-app-site-association file May 22, 2020
@jcesarmobile jcesarmobile merged commit ac9c3de into ionic-team:master May 22, 2020
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.

3 participants