numbersprotocol / capture-lite

A photo-sharing app with only verifiable photos and videos.
https://numbersprotocol.github.io/
GNU General Public License v3.0
25 stars 6 forks source link

[issue] Ionic App cannot detect iframe address correctly #1913

Open sync-by-unito[bot] opened 2 years ago

sync-by-unito[bot] commented 2 years ago

┆Issue is synchronized with this Asana task by Unito ┆Created By: Kenny Hung

sync-by-unito[bot] commented 2 years ago

➤ Kenny Hung commented:

  1. In dicover page then enter to some prod page
    1. press back button and it load my asset page directly. (It seems to load my first asset.)

right path should be

  1. In dicover page then enter to some prod page
  2. press back button and back to discover page
sync-by-unito[bot] commented 2 years ago

➤ Sam commented:

Kenny Hung, now I understand this issue. It's weird.

Ethan Wu, I have a question, how back button inside the iframe works? How does it navigate back? Is it something like window.location.back() or ? When I click back button inside iframe it somehow affects ionic route.

Here is the claap ( https://app.claap.io/numbers-protocol/dev-and-qa-only-iframe-back-button-issue-c-O35CsUM4Uy-6aNyr1FFT6Hx ) of this issue.

sync-by-unito[bot] commented 2 years ago

➤ Ethan Wu commented:

Sam the back button in iframe works by returning the previous page.

for example if they were on the explorer originally and than navigated to product page. back will take them to explorer

if they were on profile page and than navigated to product page. back will take them to profile page.

sync-by-unito[bot] commented 2 years ago

➤ Sam commented:

Ethan Wu, yes should work as you described capture app (web), but for some reason it behaves differently when it's capture app (mobile version iOS/Android).

Here is the demo ( https://app.claap.io/numbers-protocol/dev-and-qa-only-iframe-back-button-issue-part-2-c-O35CsUM4Uy-_SkCtAUvvQWQ ). Will research further. Probably will contact ionic team.

sync-by-unito[bot] commented 2 years ago

➤ Kenny Hung commented:

Additional information It seems happened on iOS.

sync-by-unito[bot] commented 2 years ago

➤ Sam commented:

asked ( https://ionic.zendesk.com/hc/en-us/requests/43040 ) ionic team about the issue. Waiting for response.

sync-by-unito[bot] commented 2 years ago

➤ Ethan Wu commented:

same as other issue card.

we will disable back button (hide it) and Sam will use explorer tab button to reset navigation.

sync-by-unito[bot] commented 2 years ago

➤ Ethan Wu commented:

for iframe: I have hidden / disabled back button

sync-by-unito[bot] commented 2 years ago

➤ Tammy Yang commented:

Ethan Wu I will turn this task into another issue Sam mentioned today. In order to allow Ionic App to detect the page navigation, we need to enable CORS. However, according to the forum https://forum.bubble.io/t/how-to-handle-cors-cross-origin-and-embed-bubbleapp/109455 ( https://forum.bubble.io/t/how-to-handle-cors-cross-origin-and-embed-bubbleapp/109455 ), it seems that this is not yet supported.

We may need to discuss with Bubble team again 😅

sync-by-unito[bot] commented 2 years ago

➤ Ethan Wu commented:

Tammy Yang I can send them another support ticket :P

sync-by-unito[bot] commented 2 years ago

➤ Ethan Wu commented:

support ticket sent.

sync-by-unito[bot] commented 2 years ago

➤ Ethan Wu commented:

Update: they received my ticket and are researching the issue for me.

sync-by-unito[bot] commented 2 years ago

➤ Ethan Wu commented:

I have sent another support ticket to bubble as the previous support has not been informative.

sync-by-unito[bot] commented 2 years ago

➤ Ethan Wu commented:

current response:

Thanks so much for following up. Yes, what you're saying is correct - this is currently a feature request. However, its one that we receive quite often, so I'm hoping our product team will prioritize this in future roadmaps. I'll be personally advocating for it as well, since I do believe it would open up a lot of possibilities on Bubble.

All of that said, I don't have a concrete timeline on when you may be able to expect a feature like this, and unfortunately, there are currently no alternative ways to modify this header information directly. Any alternative approaches that exist in your case would be referenced in the documentation of the service you are trying to integrate as alternative methods for integration.

I'm sorry for the limited action I'm able to take here, but I do hope everything I'm saying makes sense - let me know if you have any specific questions, and of course, if anything else comes up don't hesitate to reach out!

I hope you have a wonderful weekend.

sync-by-unito[bot] commented 2 years ago

➤ Ethan Wu commented:

^^ summary. this is a request that they get a lot. they are advocating for it but timeline for feature is not concrete (no information)

sync-by-unito[bot] commented 2 years ago

➤ Tammy Yang commented:

Follow up in the next sprint

sync-by-unito[bot] commented 2 years ago

➤ Ethan Wu commented:

experiment has not happened. this will be delayed. need to discuss more details with sam.

Kenny Hung ^^

sync-by-unito[bot] commented 2 years ago

➤ Kenny Hung commented:

Ethan Wu (cc Tammy YangSam) Okay, this task drop & move to product backlog.

sync-by-unito[bot] commented 2 years ago

➤ Sam commented:

Is this issue still happening?