-
Notifications
You must be signed in to change notification settings - Fork 319
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
My CC BY-NC-SA licence is not detected (MIT displayed) #175
Comments
Thanks for the report. Give me a moment and I'll try to see what's going on... CONFIRMED but the JavaScript routine is working well. I think there may be a glitch in the DB storage cached values @sizzlemctwizzle and should probably be regenerated with the changes from both #174 and #161 ... thankfully I don't think there are any more upmixes to be performed at this time. @jesus2099 |
OK great, my scripts will automatically update correctly as this one that I have updated in github today. :) |
You can't lose the webhook. You'd have to remove it from the repo. What do
|
Not sure... you might try resending your most recent webhook events... that might trigger a refresh on OUJS... same should go with resyncing it via reimporting with more of an assumed chance of refreshing. Please let us know the results. :) |
I clicked Edit scripts then Submit code on that one, it worked. I hope the webhook will still update that script if i ever change it on github. |
If you want to just reimport all your scripts back in... that should be the easiest if you don't know how to resend the webhook. |
Yes the webhook will continue to work, even if you update a script on the Essentially the responsible code receives a list of files you modified on
|
You'll probably still need to write something to fix (reconcile) the cached metadata base for those users that are pasting/uploading ... that is out of my league right now esp. without known access. Assigning to you. The longer we wait on this the more chance that #174 will generate this issue down the line too... so tagged that one with "sooner". Since we're still relatively low on script counts compared to USO I don't think it will take long. @cletusc has a pending sync commit that may be affected too. |
New metadata overwrites the old. The metadata is what we got from the current version of the script on the site. They should always match. |
Firstly they don't because of the routine that was imported from CI... it splits on spaces ( Secondlly the "upmix", as I word it, actually does minimal transformation to certain key names e.g. The meta routine is however "nearly identical" :)
Self correcting is a good thing. :) You don't have to correct the DB but it will probably be an annoyance for those experiencing this... We could just tell 'em to resync, resend webhook, or update their scripts to fix it... e.g. put the ball in their court. Based on this info what do you want me to do? Close this issue or rewrite a potentially CPU/MEM intensive merge routine (this is a reason why I opted out of it in CI because that script already eats a lot of cycles and upmixing is definitely the faster track) ?? Btw these upmixes currently only affect approximately 6.9% of our Script Authors or higher leveled user base... future update/uploads/pastes/syncs of course won't be affected. :) |
Tell users to re-upload (or whatever appropriate) their scripts if they |
Okie dokie. Closing. |
Is telling all users easier ? Don’t forget to do so. :) |
We can post a message on the site.
|
Matter of perspective. ;)
It's these users scripts from a manual audit (Reaudited a 2nd time):
We can strike out two notifications since you are here and here is notifying @jerone if he hasn't been following... so that makes a whole 8 to notify to check. :) Not all are So only ones found with and one other that didn't have So we are down to a massive 1ish users to notify. ;) (sorry for the re-edits but verifying again and these are bound to change at some point and manually is time consuming) I do appreciate greatly your reporting @jesus2099 . :) |
Wow, thanks @Martii for finding out it’s eventually this limited ! :D |
Noted 😄 |
I see MIT licence on all my scripts although I think they are all using a CC BY-NC-SA licence in their metadata.
It was detected in userscripts.org.
The text was updated successfully, but these errors were encountered: