Skip to content
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

Allowing upload of images only #338

Open
5 tasks
IshaGupta18 opened this issue Feb 11, 2019 · 37 comments
Open
5 tasks

Allowing upload of images only #338

IshaGupta18 opened this issue Feb 11, 2019 · 37 comments

Comments

@IshaGupta18
Copy link

Hi, this is a first-timers-only issue. This means we've worked to make it more legible to folks who either haven't contributed to our codebase before, or even folks who haven't contributed to open source before.

If that's you, we're interested in helping you take the first step and can answer questions and help you out as you do. Note that we're especially interested in contributions from people from groups underrepresented in free and open source software!

We know that the process of creating a pull request is the biggest barrier for new contributors. This issue is for you 💝

If you have contributed before, consider leaving this one for someone new, and looking through our general help wanted issues. Thanks!

🤔 What you will need to know.

Nothing. This issue is meant to welcome you to Open Source :) We are happy to walk you through the process.

Problem

Multiple types of file can be uploaded in the upload image section.

📋Solution

Change this line:

<i class="fa fa-plus-circle"></i> Select images <input id="warpable_uploaded_data" name="warpable[uploaded_data]" type="file" multiple />

To:

<i class="fa fa-plus-circle"></i> Select images <input id="warpable_uploaded_data" name="warpable[uploaded_data]" type="file" accept="image/*" />

  • 🙋 Claim this issue: Comment below. If someone else has claimed it, ask if they've opened a pull request already and if they're stuck -- maybe you can help them solve a problem or move it along!

  • 📝 Update

  • 💾 Commit your changes

  • 🔀 Start a Pull Request. There are two ways how you can start a pull request:

  1. If you are familiar with the terminal or would like to learn it, here is a great tutorial on how to send a pull request using the terminal.

  2. You can also edit files directly in your browser and open a pull request from there.

  • 🏁 Done Ask in comments for a review :)

🤔❓ Questions?

Leave a comment below!

Is someone else already working on this?

We encourage you to link to this issue by mentioning the issue # in your pull request, so we can see if someone's already started on it. If someone seem stuck, offer them some help! Otherwise, take a look at some other issues you can help with. Thanks!

@welcome
Copy link

welcome bot commented Feb 11, 2019

Thanks for opening your first issue here! Please follow the issue template to help us help you 👍🎉😄
If you have screenshots to share demonstrating the issue, that's really helpful! 📸 You can make a gif too!

@Jordenng
Copy link
Contributor

hey :) may i take this?

@grvsachdeva
Copy link
Member

Hi @Marieram, you can claim this one. You can see #329 for more details and ask @IshaGupta18 or me for any help. Thanks!

@IshaGupta18
Copy link
Author

IshaGupta18 commented Feb 11, 2019 via email

@Jordenng
Copy link
Contributor

Hi @Marieram, you can claim this one. You can see #329 for more details and ask @IshaGupta18 or me for any help. Thanks!

do you mean me? because you tagged someone else. thanks anyway :)

@grvsachdeva
Copy link
Member

What a timing 😅. you can surely claim this one @Jordenng. Thanks!

@IshaGupta18 I am looking for two FTOs for @Marieram and @ctam853, they are also looking for a issue to take a dig, so please tag them if you see any such opportunity. Thanks!

@IshaGupta18
Copy link
Author

@gauravano I sure will if I find any!

@marielenna
Copy link

Thank you for your efforts @gauravano !

@panhabot
Copy link

May I claim this?

@grvsachdeva
Copy link
Member

Hi @panhabot, this issue is claimed by @Jordenng. But, we would love to have your help, you can find more issues at https://code.publiclab.org/#r=all and let us know by commenting below if you can't find any. Thanks!

@panhabot
Copy link

@Jordenng Thank so much for the link!

@grvsachdeva
Copy link
Member

@panhabot I just created some FTO's at https://github.com/publiclab/plots2/. If you are still looking for issues to work on, then claim this one - publiclab/plots2#4821. Thanks!

@IshaGupta18
Copy link
Author

@Jordenng how's the work going on this one? Just checking in to see if you need any help!

@panhabot
Copy link

@gauravano sorry I replied a bit late. I think someone has already claimed it. Thank!

@grvsachdeva
Copy link
Member

No issue @panhabot. I will mention you again if I see/create a FTO candidate. Thanks!

@panhabot
Copy link

@gauravano Thank you

@meet2410shah
Copy link

I also want to claim this issue.

@IshaGupta18
Copy link
Author

@meet2410shah this issue has already been claimed by @Jordenng. If he isn't working on this, then you can take this up.

@meet2410shah
Copy link

Yes, Sure! I would like to solve this issue.

@IshaGupta18
Copy link
Author

Yeah, but let's wait for @Jordenng . @Jordenng are you still working on this one?

@meet2410shah
Copy link

Okay! Please can you suggest another issue which is currently not claimed by anyone?

@Jordenng
Copy link
Contributor

Jordenng commented Feb 21, 2019 via email

@Jordenng
Copy link
Contributor

Jordenng commented Feb 21, 2019 via email

@Jordenng
Copy link
Contributor

hey @IshaGupta18 I'm done - can you review please? :)

@IshaGupta18
Copy link
Author

@Jordenng could you please give me the link to your Pull Request?

@IshaGupta18
Copy link
Author

Okay, so I saw that you made a Pull request in your own local repository. You are supposed to make a PR here, in this repository. Does this make sense?

@Jordenng
Copy link
Contributor

sorry, did I get it right now? @IshaGupta18

@grvsachdeva
Copy link
Member

Hey @Jordenng, please add Fixes #338 in your PR's main comment to tag this issue, it will help other users to refer your PR easily and also closes this issue automatically when your PR gets merged(due to Fixes keyword).

Thanks!

@IshaGupta18
Copy link
Author

I am afraid, not @Jordenng, could you please look up on how to make a PR in the main repo? We need a PR in this repo for it to be merged.

@vishakhanihore
Copy link

@gauravano Is it free to claim ?

@kislay-kashyap
Copy link

I would love to do this. Can I?

@jywarren
Copy link
Member

jywarren commented May 7, 2020

Sure, please do! Thanks!

@colintalex
Copy link

Is anyone touching this right now? I'd love to actually get this finished.

@karimabuseer
Copy link

Hey heroes! Has anyone claimed this issue yet? If not I'd love to :)

@Xuewei-Chen
Copy link

Hi, can I claim this issue?

@VictoriaAjala
Copy link

Hello everyone, I noticed this issue has not been solved/closed yet. However, I'd love to work on this immediately if I'm permitted to. Thank you! @jywarren @IshaGupta18 @grvsachdeva

@ghost ghost mentioned this issue Aug 30, 2023
5 tasks
@ghost
Copy link

ghost commented Aug 30, 2023

Please @IshaGupta18 please check out the code.
Written as you asked to change it to

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.