Remove save code from screenshots
Categories
(Firefox :: Screenshots, task, P3)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox90 | --- | fixed |
People
(Reporter: emmamalysz, Assigned: u682976)
References
(Blocks 1 open bug)
Details
(Whiteboard: [outreachy-2021-screenshots])
Attachments
(1 file, 2 obsolete files)
+++ This bug was initially created as a clone of Bug #1705744 +++
Saving a screenshot is no longer supported and can be removed from screenshots.
Please ensure screenshots still work when submitting a patch
|
Reporter | |
Comment 2•3 years ago
|
||
Sure! Please add emalysz and sfoster as reviewers for your screenshots patches
|
||
Comment 3•3 years ago
|
||
hyy. the bug seems unassigned to me. can I work on itt?
@dawit are you working on it? if yes, please do inform me and make yourself an assignee.
Hello. Yes I'm working on this issue. Just hadn't had the time to finish it yet. I don't know if it is possible to make myself an assignee though. Can you help with this please?
|
|
Reporter | |
Comment 5•3 years ago
|
||
Hi Dawit,
You can do this by "Edit bug" --> "Assignee" and type your username. I went ahead and did it for you, but for the future, please assign yourself to any bugs you're actively working on. Thanks for taking it!
Sorry Dawit .emalysz only told you can submit anything there is nothing with the assignee . And I have been working on this a week ago.
Thanks
|
Assignee | |
Comment 10•3 years ago
|
||
You should have at least said something about it. It took me a long time to find beginner-friendly tasks on this project. I was glad this was assigned to me, and I spend multiple hours trying to figure it out. That's not very professional.
|
||
Comment 11•3 years ago
|
||
Huhh see.. all the issues are kinda 'submit a patch and make urself an assignee' type . Then this kinda things are familiarly happening, I found some dead codes in my shots files and even removed in my working directory waiting for the commit then what happened someone committed the same codes before a few minutes to my commit that I've been tirelessly found and removed . You know how sad it was . and Im really really sorry for this.If its hurts you. Hoping u understand the situation.
Thanks
|
|
Reporter | |
Comment 12•3 years ago
|
||
Hi everyone,
Thanks for both taking the time to look at this issue! Dhanesh, if an issue is assigned, that means the bug is already being worked on by another contributor/employee even if there is no patch associated with it yet. If there has been no activity on a bug for a few months, it's good practice to needinfo? the curreent assignee to see if they have any intention of picking up the bug again.
That being said, this issue is still assigned to Dawit, who should still have the opportunity to submit a patch. For that reason, I'm going to mark Dhanesh's patch as "obsolete".
This was a totally understandable mistake getting involved into a new project and is part of the learning curve. It's great practice to start looking at the screenshots code!
|
|
Reporter | |
Updated•3 years ago
|
|
||
Updated•3 years ago
|
|
|
Reporter | |
Updated•3 years ago
|
|
|
Assignee | |
Comment 13•3 years ago
|
||
|
|
Assignee | |
Comment 14•3 years ago
|
||
|
|
Assignee | |
Comment 15•3 years ago
|
||
I made a mistake when I was working on the task and submitted the patch. I didn't know how to edit a patch, so I resubmitted and new one. The former one is invalid.
|
|
||
Updated•3 years ago
|
|
||
Comment 16•3 years ago
|
||
Pushed by emalysz@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7b3f157b8b55 Remove save code from screenshots. r=emalysz
|
||
Comment 17•3 years ago
|
||
| bugherder | ||
Description
•