Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

PWA loads a 404 on Safari iOS 13.2 #46

Open
kyleschmolze opened this issue Sep 17, 2020 · 4 comments
Open

PWA loads a 404 on Safari iOS 13.2 #46

kyleschmolze opened this issue Sep 17, 2020 · 4 comments

Comments

@kyleschmolze
Copy link

On Safari, iOS 13.2, I can't seem to successfully install the PWA on my Home Screen. If I navigate to the web app (https://not911.nyc/app/) and then attempt to Share > Add to Home Screen, Safari tries to add a PWA using this URL: https://not911.nyc/app/app/ (which is then a 404).

I can add the PWA through Safari from https://not911.nyc/ directly, but then the PWA opens the marketing page, not the web app.

I've put screenshots from my phone for both of these cases below.

Might this be caused by this line in the manifest.json? Using ./app may mean "current directory plus /app", which would explain the /app/app I'm seeing.

A proposed solution would be to simply change the ./app/ to /app/, like this:

  "start_url": "/app/",

so that the current directory is overwritten. Another solution which may be the safest could be to just hard-code the start_url, which is supposedly supported, according to WebMDN. That would look like:

  "start_url": "https://not911.nyc/app",

This all said, I tried reproducing the bug in development, and I can't 😕 I can't even get my changes in manifest.json to affect the URL for the PWA in dev. So I might be way off base here on the solution. If so, sorry for the misdirection 😅 But I am getting the 404 in production.

Screenshots:

@rossPatton
Copy link
Member

Thanks for the issue! We're aware that some folks have had issues with the PWA and we're gonna look into it!

@rossPatton
Copy link
Member

So just to follow-up: I've pushed a bunch of changes and tweaks today, but wasn't able to get to this just yet. It looks like a fairly easy fix if your suggestion pans out. I will try to get to it tomorrow - but if not, you are more than welcome to PR a fix yourself!

@kyleschmolze
Copy link
Author

Yeah I did try to get it working locally and attempt the fix, but I didn't get any changed behavior from my suggestion. I didn't have the time to keep digging - could be that I was editing the wrong spot, or could be that my dev environment isn't set up propertly, or could be that I just don't fully understand the manifest.json yet!

@rossPatton
Copy link
Member

unfortunately i think this might still be an issue. i took a long break from this project, but am now working on it again more regularly. i will look into it again soon

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants