-
Notifications
You must be signed in to change notification settings - Fork 22
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
Imgur Upload #31
Comments
yes, makes sense |
Should the image be uploaded anonymously to Imgur, or should it be tied to an Imgur account? |
I think as a phase 1 it could be anonymous only and phase 2 you could implement the Imgur API and allow logging in. |
I probably missed something, but here are links to phase 1 and phase 2 Imgur integration: Phase 1: https://github.com/jkayani/a-s-minus/tree/imgur-phase1 Test them out, and if they seem pretty bug-free, I'll open a pull request. |
I was just talking about phases cause it seemed like it was going to be a lot of work. |
I thought it was a good idea since I didn't really plan on doing it all anyway; plus I guess it's better practice that they're on separate branches. Anyway, let me know if you find a flaw in there. |
I got an error with the phase1 branch with HEAD abf0773:
I also got an error while trying to use the upload:
|
The phase2 branch at HEAD ff9d386 works successfully but I got the same permission error about |
And personally, I feel like both branches should be merged together to create a single "Imgur Upload" feature branch, but I'm not a maintainer so do as you wish. |
I was able to reproduce both issues in Opera on my laptop - the issue with the The problem you found with the phase1 branch was a mistake I left behind in the code when I was working on it last; I'll make a commit to fix it soon. You'll get the same error with The phase2 branch contains the functionality of the phase1 branch, so if any branch is going to merged into the project, it should be |
I look forward to the next round of testing. |
sorry, extremely busy with other stuff... but there might be some development here in a couple weeks. |
@rojer still hasn't released it yet though. |
I cannot login to Imgur using OAuth, only login/password working... |
I can reproduce your issue @marsjaninzmarsa, but I'm not sure what causes it nor how to fix it. The login prompt appears when we use the extension API I think this issue can be closed too. |
I'd like to request a feature for being able to upload to Imgur from the extension.
The text was updated successfully, but these errors were encountered: