Importing remote, or any JSON #132
Comments
|
I've been having this problem for a couple months too, in both Chrome and Firefox. My custom comments went away, I said "huh that's odd" and tried reimporting, and the import spins forever without completing. |
|
I hope I'm not stepping on the feet of other contributors, but I think, unless you're opening the PR yourself, there are unlikely going to be any fixes anytime soon. I'd be happy to help anyone getting started with the code base. |
|
If there are features the contributers can't support, then take them out? That much should be an easy and quick task. One import and export method is quite enough (assuming it's well-documented); the rest is convenience. But the one should work reliably, otherwise the script is nearly useless. :/ |
Feel free to submit a PR. |
|
I admit to having some difficulty getting up motivation (and time) to look
into this, though now having several people reporting problems maybe I
should...
…On 15 May 2017 at 11:29, Oliver Salzburg ***@***.***> wrote:
I hope I'm not stepping on the feet of other contributors, but I think,
unless you're opening the PR yourself, there are unlikely going to be any
fixes anytime soon. I'd be happy to help anyone getting started with the
code base.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#132 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AARPAkfxhIxMU2XBQraQdHDDtR5FZb4aks5r6BsKgaJpZM4M9jVx>
.
|
|
In reply to Raphael, it's actually the jsonp format which is 'new'. The
internal format is markdown, the jsonp function was just a convenience to
allow people to write the comments in markdown then convert to jsonp for
remote.
I'll give it a few minutes and see if I can find anything. No promises..
…On 15 May 2017 at 12:15, Ben Simkins ***@***.***> wrote:
I admit to having some difficulty getting up motivation (and time) to look
into this, though now having several people reporting problems maybe I
should...
On 15 May 2017 at 11:29, Oliver Salzburg ***@***.***> wrote:
> I hope I'm not stepping on the feet of other contributors, but I think,
> unless you're opening the PR yourself, there are unlikely going to be any
> fixes anytime soon. I'd be happy to help anyone getting started with the
> code base.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <#132 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AARPAkfxhIxMU2XBQraQdHDDtR5FZb4aks5r6BsKgaJpZM4M9jVx>
> .
>
|
|
(Sorry, stream of conciousness debugging here) There's a link which sends me here, which is as clear as mud to me. This on Stack Overflow makes me wonder if the problem isn't at github's end (in this particular case). Of course all of this may be complete rubbish and related to firewall problems where I am right now! Hoping maybe my five minutes will help move things forwards a bit. |
|
@oliversalzburg I came here because the post on Stack Apps (https://stackapps.com/q/2116/21539) directs people here for bug reports. That post was my entire context for this repo before coming here. I didn't know that bug reports were only welcome with accompanying PRs. If the guidance has changed, could someone update that post (or let me know and I'll submit an edit)? If converting to another format fixes the problem, that works for me -- I'm only using jsonp because that's what the doc over on SE says. I'm just looking for a way to synchronize my comment templates across several devices and browsers. @Benjol thanks for your work on this tool. |
All bug reports are welcome. It's just unlikely that they will be handled by the original authors anytime soon. So, if you want to make sure that a fix becomes available ASAP, submitting a PR helps :) |
|
Ok, got it. Not equipped to provide the PR, alas. |
|
@cellio The remote issue won't be solved by using Markdown, I think. But the manual import works, at least. |
|
In fairness to myself, Oliver and others, the goalposts keep changing:
notably when browsers become more strict, or their plugin policies mutate.
(@Shog9 was kind enough to submit a PR the last time Stack Exchange broke
the script)
My day job hasn't involved web for a long time now, and I'm not in the
rockstar category of people who carry on working when they get home :)
…On 15 May 2017 at 15:20, cellio ***@***.***> wrote:
Ok, got it. Not equipped to provide the PR, alas.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#132 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AARPAueAF15iVOAkOtKpqgzoOR32WVtdks5r6FEigaJpZM4M9jVx>
.
|
|
Yeah, I'm not criticizing anybody for not keeping up with changes in the web ecosystem. If you thought I was, please accept my apologies for the miscommunication. I appreciate the work of people who volunteer their time and effort to make improvements like this script. |
|
No worries, I didn't read your comment that way, was just generally bemoaning the mismatch between what I'd like to be able to do and what I can do.
…-----Original Message-----
From: "cellio" <[email protected]>
Sent: 15/05/2017 17:10
To: "Benjol/SE-AutoReviewComments" <[email protected]>
Cc: "Benjol" <[email protected]>; "Mention" <[email protected]>
Subject: Re: [Benjol/SE-AutoReviewComments] Importing remote, or any JSON(#132)
Yeah, I'm not criticizing anybody for not keeping up with changes in the web ecosystem. If you thought I was, please accept my apologies for the miscommunication. I appreciate the work of people who volunteer their time and effort to make improvements like this script.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
|
Could someone try importing from somewhere other than github? |
|
I don't store mine on GitHub: |
|
@cellio, now this IS weird, because I just tried yours and they worked?! (I clicked the import "get now" link). Can you try again? |
|
@Benjol hmm, that's weird! It's still spinning from me (Chrome and Firefox, Mac 10.12). Just in case there's something weird in the JSON for just one of them, I tried yodeya.jsonp last week and then manually added them, and today I tested with writers.jsonp. |
|
@cellio, I tried again, it didn't work :( Just read this on StackApps (comments on this answer), if someone wants to try. "Just in case anyone else is looking to do this, if you want to store your JSONP on GitHub you will need to use the file extension .js and then use RawGit to serve the content up using the correct, executable, mime type." Note that this problem is going to be complicated (or may even be caused?) by the fact that SE have switched to https. #134 |
|
Any idea at all why the custom comments are frequently forgotten? For me, this happens about twice a week, on multiple machines (with Firefox 56 on Linux as well as Waterfox on Windows), and I've got to copy-paste from my Gist then (auto-pull doesn't work, as reported above). To clarify: the auto-import not working I can live with. What bothers me is the script's permanent Alzheimers. |
|
Is this a new problem?
GDPR changes maybe?
It would be interesting to know if others are experiencing the same thing
…-----Original Message-----
From: "Izzy" <[email protected]>
Sent: 18/05/2018 23:10
To: "Benjol/SE-AutoReviewComments" <[email protected]>
Cc: "Benjol" <[email protected]>; "Mention" <[email protected]>
Subject: Re: [Benjol/SE-AutoReviewComments] Importing remote, or any JSON(#132)
Any idea at all why the custom comments are frequently forgotten? For me, this happens about twice a week, on multiple machines, and I've got to copy-paste from my Gist then (auto-pull doesn't work, as reported above). To clarify: the auto-import not working I can live with. What bothers me is the script's permanent Alzheimers.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
|
@Benjol I don't remember what time it started. Unless you did some GDPR changes, I doubt it's that. I rather suspected some FF changes (v55+), but I'm not sure. In the interim I thought it might be my cookie policy (for some reason I had permitted session cookies only), but even after allowing all cookies, the issue persists. Just had it again yesterday. And it's hard to trace – as when they're gone it's already to late, and before that you can't tell when it happens next. Not sure if it's a general issue with localStorage either, as I'm not aware of something else I use which uses that. |
|
As Chrome says in response to the request:
This is not going to work because Gist provides the data as https://stackoverflow.com/questions/39228657/disable-chrome-strict-mime-type-checking |
|
@oliversalzburg OK, that explains the import part. But not why the data is frequently forgotten. What does purge it from localStorage? Any idea? |
|
@oliversalzburg could one of the answers to how to create a gist in github that returns json data work? |
|
To my understanding, LocalStorage is cleared if you clear your browser history. Of course the details depend on browser, version, clear options, but there is always the potential to also clear the items stored in LocalStorage for a site. I would assume this is a common reason for the observed behavior. Additionally, Stack Overflow sites have moved to HTTPS a while ago, which made them new sites from the browsers perspective. I assume some reports of this behavior are due to this switch. Regarding the approach from the question you linked, I guess anything that results in the proper MIME type on the response will work just fine. As the answers suggest, rawgit allows for this, so that should work fine. |
That's one possibility (doesn't match the occurences here, as I did no such thing). A second possibility is they are bound to the cookie policy. So e.g. SDC (Self Destructing Cookies; pre-FF57 Firefox addon) purges localStorage along with the cookies if they expire – which is why I mentioned that above. Should no longer be the case here as I've set the cookies for "my SE sites" to never expire (unless the server requests that). And as far as I understood this script, it should store them with the 1st party (just checked: it does).
That counts for one time, and then I was aware of that. But after that, it had happened multiple times again already (hm, though not before, now that you mention it). Concerning Rawgit: I might try that next time, cloning one of my Gists to a |
|
The only thing I can add is that we also use LocalStorage extensively in our company products and I am not aware of a single case where the data stored there "cleared itself". Maybe some script on the sites clears the information stored there. Maybe they clear it on logout. Was this noticed in non-Firefox browsers too? |
That would mean everybody using this script must have this issue. I don't know who does, so I cannot ask for confirmation. The only item I cannot match to ARC in localStorage is one with the key
I cannot tell, not using any. |
|
I have not been experiencing forgotten comments. Using Firefox, currently 60.0.2. |
|
Thanks, @reitzig – so it's unlikely to be triggered by some SE cleanup script. I have this symptom with two browers: Waterfox 56.2 (also had it with 56.1 before) on Win7, and Firefox 55.* on Linux (which I haven't yet replaced by Waterfox). |
|
Come to think of it, I also use SOUP, which may or may not influence how SE interacts with AutoReviewComments. |
|
I've just had it again. Wish there was a log to check for the culprit. |
|
@IzzySoft I haven't recently noticed comments disappearing. I haven't tried auto-update in ages because of the earlier comments; when I change something I just re-import. (And I'm using markdown now instead of json.) I'm using Firefox (which I just checked) and Chrome (which I'll verify again later). I also use SOUP. |
|
Here you go! Greatest SO userscript, period. |
|
Thanks! Installed on my Linux machine. Though I couldn't find out what exactly it does. Guess I'll learn that shortly, at least part of it 😁 |
|
Unfortunately SOUP doesn't fix it for me. AutoReviewComment is still "forgetting" my comments about once a week – but always only for one SE site at a time. Need to take a closer look at when this happens. Right now, it was on the first call after a browser restart. Not sure if it was the same in all other cases. This SO post says:
In my case, neither a nor b are met. No idea what c refers to. But it might well be that one cookie has expired and got cleared/refreshed. (Note to self: if it's cookies, next on WF should be ASE on August 1st (sgt)) |
|
@IzzySoft Sorry. It doesnt anymore. deleting my comment so that it wont distract users. |
|
Hi, i am having trouble getting my custom remote jsonp for blender se: https://raw.githubusercontent.com/L0Lock/BlenderSE-ProFormaComments/master/comments.jsonp It just shows a red "error" next to auto-get and doesn't import anything. I am no web expert so I don't know much where to look up for logs and whatnot. It doesn't show anything new in the console when running, but there is this "issue" that seems related to my comments file:
|
|
I'm very sorry, but this hasn't been maintained in years. |
|
Also as one of the last maintainers of this, I can say I have no current interest in it. Stack Exchange has long since ceased to be the community oriented effort it once was and now organizes entirely around what the parent company can make a buck off of. I'm always interested in collaborating with people on things that mutually benefit each other but have no interest in being manipulated and taken advantage of as a means of corporate profit. I still land on the site sometimes when searching for technical answers but have very little interest in contributing and will not be doing any development work on this for the foreseeable future. Others are welcome to fork and run with it but don't hold your breath for anything from me. |
|
What @alerque said. I can only second this. I happily contribute to community projects to the benefit of all – but don't like to have my work exploited for single-sided profit. So like Caleb, I mostly stopped contributing there. |
|
I also left SE when their mistreatment of contributors became too much to bear. I joined the Codidact project to build a community-first platform that's FOSS and protected by a non-profit org (so it legally cannot go the way SE did). We're small but trying to grow -- see my GitHub profile for repos/links if interested. I don't want to be spammy; if anyone finds this comment objectionable, please let me know and I'll delete it. (It sounded like some of you might be interested.) |
|
I understand, no worries. Thanks for your commitment and your hard work so far. Maybe this could be clearly stated on the readme, and/or the repo archived, just for good communication measure. And I'll sure check your repos @cellio ;) |
I used to be able to use
as remote URL and get the JSON from there (Firefox). Now the user script forgot my custom comments so I wanted to re-load them, but it does not work any longer, even with all script blockers deactivated. Maybe related to #39? Used to work after 2014, though.
I also can't seem to be able to import the JSON by copy-pasting it (clicking "jsonp" in the import/export tool will only give me the current set as export, but there's no "save"); do I really have to reformat it all into the (new?) Markdown format for manual import?
How can I get the remote source to work again?
Workaround: Convert to the Markdown format and import manually.
The text was updated successfully, but these errors were encountered: