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
Mysterious error message "User owmloading not found" shown #5095
Mysterious error message "User owmloading not found" shown #5095
Comments
@gauravano @jywarren I want to look into this next. |
Can someone explain to me what this bug is? |
What does the error mean? |
That would be great @madeofhuman! |
@harshkhandeparkar Actually, this alert is shown in some instances, mostly in login flow. The reason for this is still not known. |
Uh ok. I was also not able to upload imgs. What could be the reason? |
Locally or in prod?
…On Wed, Mar 20, 2019 at 9:27 PM Harsh Khandeparkar ***@***.***> wrote:
Uh ok. I was also not able to upload imgs. What could be the reason?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5095 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AT6S9h2iZ51V3jR7BWcryK9wC8ESLCuxks5vYlptgaJpZM4b3NFm>
.
|
prod! |
I have good internet. 8MB/s upload |
There's an issue open for upload of the image in the comments. You faced issue there or somewhere else? |
Yes, in comments. |
Hi |
@gauravano I also couldn't upload a profile img. |
If it helps diagnose the problem, I've had this recently as well. For me, it reveals itself after clicking on a link to see a filtered list of my content from my profile. For example: Doesn't effect any functionality. Firefox 66.0.3 |
I think i got this. this is an artifact from LEL, for the "loading" gif. it may try to load this at the wrong location here: It's a relative location so that's why it shows up so many places. And you may get this notice on the next page load if this map gets loaded and it tries to fetch this .gif ! @ananyaarun @sagarpreet-chadha can we address this issue by eliminating that loading spinner and replacing it with a corrected relative path, or something similar? Maybe the ability to pass a root URL parameter on which to base relative URLs to this loading spinner? Or how does the regular spinner work? This is actually starting to cause system tests to fail, as well! Like in #5995. Whew, this was a great one to find! Now we just have to solve it :-) |
I'm glad you like it? I mean, the bugs tend to find me not the other way around. ; ) |
Wow!! this really took time. |
I think this is now solved due to changes in upstream https://github.com/publiclab/leaflet-environmental-layers/! Thanks all! |
This is now recurring again, noting relation to #7939 |
Hi @Tlazypanda -- i reopened this and I think this is also in relation to the asset locations being defined relatively... do you think that may be right? Is it something we could monitor by watching the Chrome dev console while loading a map, and seeing how the path matches or doesn't match the location of the Leaflet assets? Thank you! |
As you've been working on assets a lot, if you wouldn't mind adding this to your task list that would be super helpful!! |
If it would help I’d be happy to change my username to “owmloading” so I will always be found. LMK. : P
Actually, in all seriousness that may have enough comedic value to do.
Wait.
Nope.
It won’t be so funny if it ends causing a Little Bobby Tables situation.
https://xkcd.com/327/
|
Lol @skilfullycurled |
@skilfullycurled @ebarry That cracked me up lol 😂 @jywarren I found this on the lel live site might be of use: - investigating more to find out this ghost xD @jywarren where are assets for lel stored? because i can't seem to find this file ... on checking this link out https://github.com/buche/leaflet-openweathermap/ found out that owmloading.gif should be stored at this path https://github.com/buche/leaflet-openweathermap/blob/master/example/leaflet/owmloading.gif. Tried with the local lel repo and the 404 error is not there anymore. Should I make this changes and open a pr on lel? 😅 We can also store this in a separate directory in lel (i don't know right now what is the one for assets like this on lel) and I will reference it in a relative manner then ✌️ |
While my Ruby abilities are very limited, I do try to insert attempts (emphasis on attempts) at humor especially when it’s a bug I submitted. Not as helpful as actually tracking it down and fixing it, but hey, I’ve only got what they gave me.
|
Hi! Thanks for investigating this!!! I think we should see if this is referenced from a stylesheet, in which case we could override it with our own styles, or from JavaScript, in which case perhaps we can override it there? Can you look for the exact line of code where it happens and try to debug from there. Overriding in JavaScript may be harder; see how Leaflet has allowed people to override image paths here: https://leafletjs.com/reference-1.4.0.html#icon-default-imagepath So we could try to do something like that, either locally in our instance, or potentially filing an upstream PR to What do you think? |
I think this is a candidate for our |
@jywarren Actually on using the git blame feature (thanks for telling me about this simplifies debugging a lot :D) came to know that this line was introduced in publiclab/leaflet-environmental-layers@f38d594 where we added the Openweather map module code but this code referenced the owmloading.gif too but this wasn't added. So following the Openweather map module(https://github.com/buche/leaflet-openweathermap/ - owmloading file was present from the initial commits of this library) so this had to be added in assets/images. Once I did that, and checked the examples/index in browser the error |
👻 This just happened again. |
Just an update - reading the above again, i see that "owmloading.gif" means "open weather map loading" and not a mis-spelling of "downloading" - lol! https://github.com/buche/leaflet-openweathermap/blob/master/example/leaflet/owmloading.gif publiclab/leaflet-environmental-layers#473 attempted to add it locally within LEL, which could work. I'll try that again, but we can also configure it here: cc @RuthNjeri |
Progress at publiclab/leaflet-environmental-layers#504 once we get tests running again via GitHub actions for that repo! Then we'll bump the version and re-release. |
fix #5095 🎉 🎉 🎉 🎉 🎉 once LEL v2.4.3 merges!!!
fix #5095 🎉 🎉 🎉 🎉 🎉 once LEL v2.4.3 merges!!!
Final fix in #9208, whew. |
Wow finally!!! this is awesome 🎉 🎉 |
fix publiclab#5095 🎉 🎉 🎉 🎉 🎉 once LEL v2.4.3 merges!!!
fix publiclab#5095 🎉 🎉 🎉 🎉 🎉 once LEL v2.4.3 merges!!!
fix publiclab#5095 🎉 🎉 🎉 🎉 🎉 once LEL v2.4.3 merges!!!
I got this error when I opened my profile
I have used the website for the first time so please correct me if I have asked something wrong. Also my images were not posting on comments and even on profile image. Is this some internet problem or something else?
The text was updated successfully, but these errors were encountered: