-
Notifications
You must be signed in to change notification settings - Fork 52
Additional good first issues! #69
Comments
Can I try solving the issue #248 from "codetribute" while you check my last PR? |
@Joaquin-M2 Great to see how eager you are to solve problems! But let's hold off on starting a second contribution until #71 is reviewed. We want to give every applicant the chance to pick one issue to tackle first. |
Even though I was not assigned to the issue, thanks to my insomnia I have been lurking on the Codetribute code. Are you sure the issue #248 keeps being an issue? It looks solved to me. I checked this PR, which aims to solve the mentioned issue: mozilla-frontend-infra/codetribute#369 In order to confirm the PR was merged, I just checked the code that pretended to modify that PR:
Just for additional confirmation purposes, I doubled checked the issue was fixed by reproducing it in my local machine. I did it in both, Firefox and Chrome browsers, and I didn't get any error in the consoles:
|
@Joaquin-M2 If those have not been claimed (by anyone at all, not just the Pocket project applicants), go ahead and comment on the issue and request to work on it. |
closing, as the outreachy application process is complete. |
If you'd like to work on an issue for your Outreachy application that's not in this repo, the following may be good:
mozilla-frontend-infra/codetribute#248
taskcluster/material-ui-json-schema-viewer#116
taskcluster/material-ui-json-schema-viewer#115
If you'd like to work on any of these issues, comment here and we will coordinate.
The text was updated successfully, but these errors were encountered: