hackforla / website

Hack for LA's website
https://www.hackforla.org
GNU General Public License v2.0
300 stars 750 forks source link

ER: Events page continuously loading video #4370

Open MattPereira opened 1 year ago

MattPereira commented 1 year ago

Emergent Requirement - Problem

When you visit the events page, the client intermittently fires off POST requests involving the Jennifer Pahlka video that is displayed (embedded?) on the page. On my browser, the loading circle continues for a few minutes before stopping.

image

See screenshot ![image](https://user-images.githubusercontent.com/73561520/229371498-19a4cbf0-f1dd-4ef1-a934-06ae258c9f3b.png)

This problem appears to occur only on MacOS. See comment below for Slack discussion regarding this problem.

Issue you discovered this emergent requirement in

Date discovered

2023-04-02

Did you have to do something temporarily

Who was involved

@MattPereira

What happens if this is not addressed

Confusing user experience with continuous loading circle around favicon. Using extra bandwidth is a wasteful. If you have a slow internet connection, it can cause extra lag.

Resources

https://www.hackforla.org/events/

See screenshot ![image](https://user-images.githubusercontent.com/73561520/229368442-530ee235-e0e1-4c49-a0c9-cb4a543084ca.png)

Recommended Action Items

Potential solutions [draft]

github-actions[bot] commented 1 year ago

Hi @MattPereira.

Please don't forget to add the proper labels to this issue. Currently, the labels for the following are missing: Complexity, Role, Feature

NOTE: Please ignore the adding proper labels comment if you do not have 'write' access to this directory.

To add a label, take a look at Github's documentation here.

Also, don't forget to remove the "missing labels" afterwards. To remove a label, the process is similar to adding a label, but you select a currently added label to remove it.

After the proper labels are added, the merge team will review the issue and add a "Ready for Prioritization" label once it is ready for prioritization.

Additional Resources:

roslynwythe commented 6 months ago

roslyn wythe: Hi everyone, I need to ask for help testing. We have an ER https://github.com/hackforla/website/issues/4370 which describes a problem loading the video on the events page https://www.hackforla.org/events. In the ER, Matt reports that the loading circle continues for a few minutes before stopping. I'm not able to reproduce the problem. I do see periodic network traffic - POSTs to a server embed.ted.com which is a keep alive connection, but that does not appear to be a problem. Is anyone else able to reproduce a problem loading the video? (edited)

cnk: Yes I am seeing spinning, spinning, spinning. I am using Chrome Version 119.0.6045.199 (Official Build) on Mac 13.6.4 (Ventura). Same problem when I use Edge (a browser that I have NOT customized at all). Am seeing Tracking Prevention blocked access to storage for https://hbx.media.net/__media__/js/ucreative.js?cv=1. Is that where the vidoe file is?

cnk: events/:1 Tracking Prevention blocked access to storage for https://hbx.media.net/checksync.php?cid=8CUETUT4L&cs=1&cv=37&hb=1&vsSync=1&prvid=3%2C51%2C55%2C80%2C97%2C108%2C117%2C126%2C141%2C175%2C178%2C201%2C203%2C214%2C222%2C229%2C233%2C236%2C237%2C246%2C251%2C255%2C262%2C307%2C339%2C459%2C461%2C550%2C3007%2C3012%2C3016%2C3017%2C3018&refUrl=https%3A%2F%2Fwww.hackforla.org%2F&gdpr=0&gdprconsent=0&gdprstring=&usp_status=1&usp_consent=1&uspstring=&axid_e=&coppa=0&ckdel=0&gpp=&gpp_sid=.

cnk: I think it might be trying to show this cross site because I can use the link to www.ted.com and see the video just fine

roslyn wythe: do you know why your browser would do that?

cnk: I think it is a MacOS thing. Something about the default privacy setting

github-actions[bot] commented 6 months ago

Hi @roslynwythe, thank you for taking up this issue! Hfla appreciates you :)

Do let fellow developers know about your:- i. Availability: (When are you available to work on the issue/answer questions other programmers might have about your issue?) ii. ETA: (When do you expect this issue to be completed?)

You're awesome!

P.S. - You may not take up another issue until this issue gets merged (or closed). Thanks again :)

roslynwythe commented 6 months ago
ExperimentsInHonesty commented 6 months ago

RW will rewrite the proposed solution to clarify that the issue that needs to be written will be to find out if there is another way of embeding the video that does not use an iframe. And if that is accomplished that, it resolves the error.

roslynwythe commented 6 months ago
ExperimentsInHonesty commented 2 months ago

Create an issue to re-evaluate because the other issue is closed (that was blocking this was closed). Go read the other issue https://github.com/hackforla/website/issues/6353 and see what happened and do we still need to do anything.