toeverything / AFFiNE

There can be more than Notion and Miro. AFFiNE(pronounced [ə‘fain]) is a next-gen knowledge base that brings planning, sorting and creating all together. Privacy first, open-source, customizable and ready to use.
https://affine.pro
Other
42.08k stars 2.75k forks source link

​Blank screen on mobile when sharing #8338

Closed artwist-polyakov closed 3 weeks ago

artwist-polyakov commented 1 month ago

What happened?

On mobile browsers, when attempting to open shared notes, a blank screen is displayed.

Steps to Reproduce:

  1. Open Chrome browser on a mobile device.
  2. Go to this link. https://docs.polyakov.marketing/workspace/8f799969-8391-40eb-a85e-722cd6146c06/2wXXc3IsiRkYXavczZcUA?mode=page
  3. Observe the blank screen.

Additional Information:

logs in console: CleanShot 2024-09-20 at 18 26 57@2x

large screenshot from iPhone AFFiNE

on desktops: CleanShot 2024-09-20 at 18 17 53@2x

Distribution version

Web (app.affine.pro)

What browsers are you seeing the problem on if you're using web version?

Chrome

Are you self-hosting?

Relevant log output

No response

Anything else?

No response

affine-issue-bot[bot] commented 1 month ago

Issue Status: 🆕 *Untriaged

*🆕 Untriaged**

The team has not yet reviewed the issue. We usually do it within one business day. Docs: https://github.com/toeverything/AFFiNE/blob/canary/docs/issue-triaging.md

This is an automatic reply by the bot.

artwist-polyakov commented 1 month ago

instead of the JS and CSS code, an error page is returned: https://docs.polyakov.marketing/js/runtime.3b180e24.js https://docs.polyakov.marketing/vendor.4b03f911.css and this is the case for all mobile version resources. For comparison, the desktop script: https://docs.polyakov.marketing/js/runtime.a8387fa9.js

Most likely, the problem is with the engine, since the URLs are fundamentally the same, i.e., nginx serves them from the same place, and the issue is that the mobile ones are somehow not there. Or it could be something related to access rights.

PengMai1998 commented 1 month ago

instead of the JS and CSS code, an error page is returned: https://docs.polyakov.marketing/js/runtime.3b180e24.js https://docs.polyakov.marketing/vendor.4b03f911.css and this is the case for all mobile version resources. For comparison, the desktop script: https://docs.polyakov.marketing/js/runtime.a8387fa9.js

Most likely, the problem is with the engine, since the URLs are fundamentally the same, i.e., nginx serves them from the same place, and the issue is that the mobile ones are somehow not there. Or it could be something related to access rights.

Hello, how is this problem solved?

artwist-polyakov commented 1 month ago

instead of the JS and CSS code, an error page is returned: https://docs.polyakov.marketing/js/runtime.3b180e24.js https://docs.polyakov.marketing/vendor.4b03f911.css and this is the case for all mobile version resources. For comparison, the desktop script: https://docs.polyakov.marketing/js/runtime.a8387fa9.js Most likely, the problem is with the engine, since the URLs are fundamentally the same, i.e., nginx serves them from the same place, and the issue is that the mobile ones are somehow not there. Or it could be something related to access rights.

Hello, how is this problem solved?

It isn't solved.

artwist-polyakov commented 3 weeks ago

@forehalo hello, problem still be represented after update to last stable. Why do you close this issue?

fourdim commented 3 weeks ago

@forehalo hello, problem still be represented after update to last stable. Why do you close this issue?

It's fixed but not yet released in stable. Once linked PR is merged to the main branch, GitHub will automatically close the issue for you.

image

That's how GitHub works.

For the stable releases: For a version X.Y.Z: When X, Y increases, the release will include all the main branch commits in the normal case. When Z increases, only those urgent bug fixes will be included.

artwist-polyakov commented 3 weeks ago

@fourdim Got it! Still waiting next stable)

artwist-polyakov commented 2 weeks ago

@fourdim hello, i've just update version to last stable, and bug with mobile version of pages still being represented

fourdim commented 2 weeks ago

@fourdim hello, i've just update version to last stable, and bug with mobile version of pages still being represented

0.17.5 is still a patch release 0.18.0

ATPs commented 6 days ago

I tried to use the beta version of 0.18.0. I can view the link, but it requires to login. Use firefox will be OK for mobile view.