-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Custom picture(s) selector which indicates what pictures have been uploaded already #175
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
Comments
This comment has been minimized.
This comment has been minimized.
I created a similar tool for Linux: https://github.com/nicolas-raoul/nautilus_commons_upload_status |
This comment has been minimized.
This comment has been minimized.
Hi! I just saw an Android app project listed here: https://www.mediawiki.org/wiki/Google_Summer_of_Code/2021#Add_a_custom_picture_selector_to_the_Commons_Android_app that I couldn't identify who added. @nicolas-raoul @madhurgupta10 Could you confirm if you plan to mentor this project in the upcoming round of GSoC? If so, could you also add it to Phabricator? |
Hi Sir Nicolas, Can you guide me with some basic tasks that I should start with? Waiting for your reply. Thankyou, |
Also @nicolas-raoul I read the reviews of the commons app, and most of the people are facing a common issue of not being able to zoom in a picture, watching a video, and are having constant crashes. Can we create a project for GSoC and fix these issues too? Thank you. |
@srish Thanks for the nudge! We will add a Phabricator issue. |
@srish Yes, we are mentoring :) |
@srish I created a Phabricator task: https://phabricator.wikimedia.org/T278015 |
Please help me out. Waiting for a reply😊 |
Just wanted to mention that the zoom issue (and the majority of the crashes) are somewhat outdated - zoom has been in the app for a couple of versions now. However, video support has indeed not yet been implemented and is frequently requested, so that could be a potential topic IMO (if the mentors are okay with it). :) |
Btw, I think this is a fantastic idea @nicolas-raoul ! I would find it very helpful personally. :) However I should probably mention that when we looked at open source custom picture selector libraries a couple of years back, we had multiple issues with the ones we tried. I think there was an issue with multiple uploads and also an issue with Google Photos not working with them. (@maskaravivek @ashishkumar468 do you guys remember this?) So to assess the feasibility of this implementation, the student should probably review and try a few libraries and make sure that the above bugs don't happen, or be prepared to implement their own if a suitable one cannot be found. |
Thankyou for replying 😊
Oh that's great if those issues have been fixed.
Yeah, finding/creating library is going to be a fun but hectic work😀 |
Other than that Sir @nicolas-raoul @madhurgupta10 should I start working on the tasks (given here :https://github.com/commons-app/commons-app-documentation/blob/master/android/Students.md) ? Or wait for the application period to begin? |
@HamzaBinMubeen you can start working on the issues. It will help you get familiar with the project and codebase. |
Okayy Sure!! |
Hi Sir @madhurgupta10. |
You can use any OS at your convenience. |
@HamzaBinMubeen Are you able to build and run any other Android app, for instance https://github.com/tema6120/ForgetMeNot ? |
I am currently fresh installing the android studio, and will then try again. Will also try running other project and update you |
I have fixed the issue. There was some problem with the installation of android studio/sdk and/or JDK folder. |
If anyone has this same issue, remove all the files related to android studio (sdk/jdk etc) and restart to completely remove everything, then,
Incase SDK installation fails, check internet connection and firewall settings, and try again. |
Hi! Everyone, I am interested in working on this project and I am planning to implement a custom picture selector from scratch, Here the UI mock of what I am thinking to implement will surely improve it based on the feedback and also @misaochan can you provide more details of exactly what issue has been faced previously as you have mentioned. Thanks! |
@Pratham2305 Sorry for asking so many questions! Such questions are meant as a way to help you estimate how much time you will need for implementation/testing :-) You have to find the right balance between the number of features and your time. |
I will be using content provider to query all the images and album folders, So I guess these can be any folder with images inside internal storage or sd card, as of now I have manually inserted them in the layout just for the sake of these mockups
Yes these thumbnails will be replaced by the actual images and that "mountain-sun" thumbnail will be replaced by the most recent one from the folder
As of now, I was thinking to show all photo as the default album but it is a great suggestion will surely include it as one of the features :) |
@nicolas-raoul this is great ideas |
@vinayak0505 Interesting! Feel free to make a proposal based on this, I have a few questions though:
|
Yes , I have already modified that Library and added 2 methods setdisabled
method a disabled text method in which you can pass test to display as a
toast when user clicks on it.
|
please check pull request #4331 |
@Pratham2305 can you help me how you create this picture https://user-images.githubusercontent.com/54663429/112966954-598d7080-9168-11eb-8a6f-06f93ed31bcc.png please help me |
@Prince-kushwaha Sure, I have created it on the android studio itself, basically created a thumbnail layout, and used a simple recyclerview to populate the screen. I think you can create it in the same way and made changes whatever you like :) |
@Pratham2305 thanks |
Understood, don't worry, it is probably a CI temporary failure.
This project has existed for a long time, so some parts might not be using the best practice. Anyone with 5 merged bugfixes or more is welcome to send pull requests for this! :-)
Anyone with 5 merged bugfixes or more is welcome to concert classes from Java to Kotlin.
You must get a bugfix merged before the application deadline (which is in about a week I believe). You can not submit the bugfix afterwards, because we choose the candidate based on these bugfixes. Thanks for your understanding! |
@nicolas-raoul i want to make gsoc proporsal on this idea below is my implemention Strategy tell me how is my implemention strategory |
This idea appears to be largely identical to the previous comment at #175 (comment) Response to that idea by Nicolas could be found at #175 (comment) |
@gabrielchl i only use their photos i change the photos |
@Prince-kushwaha It looks good! Please thank Pratham for the mockups ;-) @gabrielchl That is true. Actually, this year's project is not so open-ended, it can be expected that all candidates' proposal will be rather similar, with probably only small variations. As detailed at https://github.com/commons-app/commons-app-documentation/blob/master/android/Students.md#how-we-choose-the-student, we choose the student mostly based on their bugfix, rather than by minutely comparing proposals. |
The mockups and implementation ideas under this issue looks great, thanks to everyone who collaborate here, I look forward to see proposals. Even if it is expected to have similar proposals, we prefer to see your own problem solving skills and perspective too. Also, the thing that @nicolas-raoul said is very true, we choose student mostly based on their contributions. |
Congrats one being selected in GSoC to work on this project @4D17Y4 ! 🎉 Hope you have a great experience :) |
Congratulations budy, wish you good luck |
Thanks, @sivaraam, @vinayak0505 😄. |
Congratulations @4D17Y4 ! Glad to have you on board. :) |
Glad to announce that the project is merged with master. Thank you everyone for all your inputs, would love your feedback on the Brand New Custom Selector. Thanks. |
Users can upload images from the app by tapping the "+" button of the main activity and tapping the "gallery" button which calls Android's built-in picture selector.
This issue would introduce a custom picture selector, which shows whether each picture has been uploaded to Commons already or not.
This is useful because after a long trip it is difficult to remember what pictures you have uploaded already.
The custom selector would show a gallery of thumbnails where already uploaded pictures are faded out and have a Commons logo overlay.
Sketch UI
New button to select the custom picture selector:
Custom selector:
In the example above, you can see that I have already uploaded the 4 pictures on the right, but not the two pictures on the left. Seeing this, I can decide whether I want to upload the first picture or not, given I have arguably already uploaded a better picture (with less cars).
Requirements
Bonus
Would be nice, but not strictly required:
GPS" overlay icon for pictures that do not have EXIF latitude/longitude.Strategies
API
Commons has a simple API to check the presence or not of a file, it works by sending the SHA1 checksum to the URL
https://commons.wikimedia.org/w/api.php?action=query&list=allimages&format=xml&aisha1=<put SHA1 here>
. Examples:The API is quite slow (around 1 second per request), which is why caching and asynchronous loading is needed. The selector must be usable even if Internet is extremely slow or disconnected.
The text was updated successfully, but these errors were encountered: