1525613 - Create a preference to enable message grouping in the console

Status: RESOLVED FIXED

(DevTools :: Console, enhancement, P1)

Description

[Nicolas Chevobbe [:nchevobbe]]

The preference should be set to false as it will be used for the development process.
We could name it devtools.webconsole.groupWarningMessages for example.

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Hello Helenatxu, would you be interested working on this bug?

This is a nice first bug, we need to add the following line :

pref("devtools.webconsole.groupWarningMessages", false);

in devtools/client/preferences/devtools-client.js#286.

Comment 2

[Helena Moreno (aka helenatxu)]

Hi Nicolas! Thank you for thinking about me for this bug, I will get my hands on it asap!

Comment 3

[Helena Moreno (aka helenatxu)]

(In reply to Nicolas Chevobbe from comment #1)

Hello Marco, after some problems downloading the repo, I finally managed to get everything running on my laptop.

So, that's it with this issue? Just to add that line in between all the other preferences? Shouldn't I add any other functionality anywhere?

Comment 4

[Nicolas Chevobbe [:nchevobbe]]

Hello Helena, I'm glad everything worked out, but I'm sorry that was so painful :/

So, yes, the bug is about adding this preference, which will be used after for some functionality.
For now, it will get you used to create a patch, submit to review, … in short, get familiar with the workflow :)

You can check that the line you entered has an effect by running firefox with the change, opening about:config, search for devtools.webconsole.groupWarningMessages and check that it is indeed set to false.

Then you can commit, and push to review (you can read https://docs.firefox-dev.tools/contributing/code-reviews.html for that).

Thanks!

Comment 5

[Helena Moreno (aka helenatxu)]

(In reply to Nicolas Chevobbe from comment #4)

Hi Nicolas! Thank you for assigning me an easy patch to start with so I get familiar with workflow, which was something very new to me. I have submitted already my patch here https://phabricator.services.mozilla.com/D21356

You may say that I renamed in Phabricator the title to correct a typo and add the correct description to the commit. Also the first time I did it I first executed the arc diff command before doing hg commit, so I changed some commented text to be able to execute arc diff again. Newbie mistakes :)

Could you tell me if there is anything else to do? If not, is it possible to land my patch and assign me a new bug to keep contributing? Thank you Nicolas!

Comment 6

[Helena Moreno (aka helenatxu)]

Attachment: Bug 1525613 - Create a preference to enable message grouping in the console. r=nchevobbe.

Bug 1525613 - Created a preference to enable message grouping in the console. r=nchevobbe.

Comment 7

[Nicolas Chevobbe [:nchevobbe]]

Thanks a lot Helena!
Let's wait until TRY (our test infrastructure) is done.

For the next patch, you might be interested in using https://github.com/mozilla-conduit/review to push patches to phabricator.
There's a few checks that are run (for the commit message format, the reviewer, …), and it also auto fill the relevant fields on phabricator based on that.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/23ec54c48f04
Create a preference to enable message grouping in the console. r=nchevobbe.

Comment 9

[Nicolas Chevobbe [:nchevobbe]]

Everything looked good on try, so I pushed the patch to autoland. It will run a few more tests and stuff, and at some point later today or tomorrow the bug should be marked as resolved :)

I'm going to try to file new good first bugs today, I'll ping you there :)

Comment 10

[Helena Moreno (aka helenatxu)]

Thank you Nicolas for your guidance :)

I'll wait then for the new bug to be assign to me.

Comment 11

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/23ec54c48f04