greasyfork-org / greasyfork

An online repository of user scripts.
https://greasyfork.org
GNU General Public License v3.0
1.47k stars 441 forks source link

"Post New Version of Script" Not functioning #225

Closed ricci07 closed 10 years ago

ricci07 commented 10 years ago

When I press the button with all with my new and updated descriptions for my script and for that matter, the updated code, it just doesn't keep the changes.

You press the button and it looks like it does something, but then you look at the code and description and it is the old version.

So you basically lose everything you just newly typed (in the description)

Please look in to this.

JasonBarnabe commented 10 years ago

It must be something specific to what you posted that did it, as there are many updates working fine - https://greasyfork.org/scripts?sort=updated

What's the URL of your script, and what's the new data you're posting?

ricci07 commented 10 years ago

Well for example this script: https://greasyfork.org/scripts/3937-youtube-annotation-ripper

But this happens on most scripts. I try to update a script after initial upload and it doesn't process it or something.

I am trying to make it so the so script is updated to this:

http://pastebin.com/s5akzixP

JasonBarnabe commented 10 years ago

In the log, the latest attempted update I see for that script resulted in a 200, which indicates a validation fail. I can see that the @namespace changed between your initial code and your new code, and Greasy Fork would've warned you about that and refused to save unless you aknowledged the warning. I don't see the warning-aknowledgement parameter sent.

This is what you should've seen. Notice the error message with a checkbox and the pink tone of the code box indicating an error. error

ricci07 commented 10 years ago

ahhh.. thanks, but no. I didn't get this at all. I just got a refreshed page like load. I did change the namespace on update for this one because I forgot to change it where I wrote it in Tampermonkey.

I will try to update with my newest version to and see if it happens again.

BTW, how can I change the namespace if this happens? It isn't correct on the old version.

ricci07 commented 10 years ago

I'm starting to think it is me being foolish. If it isn't too much trouble, could you give the error message a css background of something that grabs attention. or just encase the whole 'error warning block' in a tinted background div. If not, I can always make a script for that.

Something like this: http://i.imgur.com/R7n4S0u.png

Thanks for the help.

ricci07 commented 10 years ago

In fact, here it is: https://greasyfork.org/scripts/3981-easy-to-see-greasy-fork-errors

For some reason, I cannot change the image widths in the description, strange you wouldn't allow that attribute.

JasonBarnabe commented 10 years ago

Now the error messages show up in unmistakable eye-scorching yellow.

ricci07 commented 10 years ago

oo i'll have a look :)