-
Notifications
You must be signed in to change notification settings - Fork 208
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
Comments
Thanks for opening your first issue here! Please follow the issue template to help us help you 👍🎉😄 |
hey :) may i take this? |
Hi @Marieram, you can claim this one. You can see #329 for more details and ask @IshaGupta18 or me for any help. Thanks! |
Yes! Go right ahead and don't hesitate to ask for any help! Thanks a lot!
…On Tue, Feb 12, 2019, 12:21 AM Gaurav Sachdeva ***@***.*** wrote:
Hi @Marieram <https://github.com/marieram>, you can claim this one. You
can see #329 <#329> for
more details and ask @IshaGupta18 <https://github.com/IshaGupta18> or me
for any help. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#338 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/Am54Z5SNCjYf10YnOYRQdHVmYPFDVYwJks5vMbukgaJpZM4a0wu4>
.
|
do you mean me? because you tagged someone else. thanks anyway :) |
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! |
@gauravano I sure will if I find any! |
Thank you for your efforts @gauravano ! |
May I claim this? |
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! |
@Jordenng Thank so much for the link! |
@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! |
@Jordenng how's the work going on this one? Just checking in to see if you need any help! |
@gauravano sorry I replied a bit late. I think someone has already claimed it. Thank! |
No issue @panhabot. I will mention you again if I see/create a FTO candidate. Thanks! |
@gauravano Thank you |
I also want to claim this issue. |
@meet2410shah this issue has already been claimed by @Jordenng. If he isn't working on this, then you can take this up. |
Yes, Sure! I would like to solve this issue. |
Okay! Please can you suggest another issue which is currently not claimed by anyone? |
Yes:)
…On Wednesday, February 20, 2019, Meet Shah ***@***.***> wrote:
Okay! Please can you suggest another issue which is currently not claimed
by anyone?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#338 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AePNdYq_fzOwQ90p8z78ZQJbryN2q7ZSks5vPZTigaJpZM4a0wu4>
.
--
Yarden
|
I am working on it
…On Thursday, February 21, 2019, Yarden Nagel ***@***.***> wrote:
Yes:)
On Wednesday, February 20, 2019, Meet Shah ***@***.***>
wrote:
> Okay! Please can you suggest another issue which is currently not claimed
> by anyone?
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#338 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AePNdYq_fzOwQ90p8z78ZQJbryN2q7ZSks5vPZTigaJpZM4a0wu4>
> .
>
--
Yarden
--
Yarden
|
hey @IshaGupta18 I'm done - can you review please? :) |
@Jordenng could you please give me the link to your Pull Request? |
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? |
sorry, did I get it right now? @IshaGupta18 |
Hey @Jordenng, please add Thanks! |
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. |
@gauravano Is it free to claim ? |
I would love to do this. Can I? |
Sure, please do! Thanks! |
Is anyone touching this right now? I'd love to actually get this finished. |
Hey heroes! Has anyone claimed this issue yet? If not I'd love to :) |
Hi, can I claim this issue? |
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 |
Please @IshaGupta18 please check out the code. |
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:
mapknitter/app/views/images/_new.html.erb
Line 19 in de15628
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:
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.
You can also edit files directly in your browser and open a pull request from there.
🤔❓ 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!
The text was updated successfully, but these errors were encountered: