In T315930#11408147, @Kudpung wrote:

IMO it should be reviewer driven but be completely optional for reviewers. A BLP banner is a must but the effort should be to streamline the Page Curation tool for reviewers rather than add new tasks. There are literally 100s of Wikiprojects, not even NPPers know more than a few. A dropdown would be very long. There was once a suggestion for NPP to add stub tags.(I think it was mine). Same problem.

In T315930#11407520, @Novem_Linguae wrote:

Thanks for your ideas and analysis. Here's some thoughts:

• Mass adding {{Talk header}} to talk pages goes against current community consensus, if I recall correctly. So this task should probably be re-scoped to adding the WikiProject banner shell only.
• NPPers adding WikiProject tags is optional, so the modal would need to be designed in a way that it is optional, out of the way, doesn't add extra mandatory clicks if you want to skip it, or has a preference to turn it off.
• WikiProject tags and BLP detection should probably be done in separate patches, so that no one patch is too complex, which will increase the speed of reviews. We could also collect class as well. So might want to do 3 patches, starting with WikiProject tags, and then adding BLP and class if you get time.
• That proposed wikicode looks non-standard. I'd recommend using the standard formatting at https://en.wikipedia.org/wiki/Template:WikiProject_banner_shell#Basic
• I completely agree with it being reviewer-driven instead of automatic, and with possibly adding "suggested" WikiProjects later.

Hope that helps.

Hi everyone,
I'd like to add some thoughts on how this feature could be implemented, along with a possible technical approach.

@Athulvis Issue resolved.
MR: https://gitlab.wikimedia.org/jinoytommanjaly/bulk-user-info-fetcher/-/merge_requests/2

I'm working on this @Athulvis

Thanks @Jdlrobson . I'm little busy this days with LFX and college exams. I'll be back again contributing to Wikimedia in a while. Thanks for your help again!

Also i made the changes to the linter of chessViewer extension, not of mediawiki root directory.

I changed the linter to ES6, here is .eslintrc.json

@Adityakumar0545 Please check the patch checklist. Also import QUnit with respect to ES5 and fix the linting errors.

Okay i'll work on it. I have my semester exams around the corner.

I changed the tests and everything is now working fine.

i made the changes you suggested in the first patch.

I wrapped it around a div with a class, but running old tests on it is giving error. Should i also update the tests?

In T369609#10037882, @AAlhazwani-WMF wrote:

In T369609#10035885, @Anujagrawal wrote:

I'm not able to replicate the error and find the same appearance dropdown.

in order to display the appearance button you need to enable the "Accessibility for Reading (Vector 2022)" beta feature in your user preferences

Is this good?

I'm not able to access the page, also the page doesn't seem of wikipedia beta.

I'm not able to replicate the error and find the same appearance dropdown. Also i found a scope for improvement.

In T366300#10033603, @Jdlrobson wrote:

In T366300#10033220, @Anujagrawal wrote:

I tried working on it and visited http://localhost:4000/index.php?title=MediaWiki:GrowthExperimentsConfig.json&action=edit but i'm not able to see the page like above screenshot. I'll be very thankful if someone could help me.

Hi, did you edit the localhost page to contain the content of https://es.wikipedia.beta.wmflabs.org/w/index.php?title=MediaWiki:GrowthExperimentsConfig.json&action=edit ? Could you share some screenshots of how it is rendering for you?

When visiting http://localhost:8888/wiki/MediaWiki:GrowthExperimentsConfig.json?action=info please make sure it says
Page content model JSON (change)

@Wugapodes

I tried working on it and visited http://localhost:4000/index.php?title=MediaWiki:GrowthExperimentsConfig.json&action=edit but i'm not able to see the page like above screenshot. I'll be very thankful if someone could help me.

should i change the tests to ES5 or change linter to ES6?

@Wugapodes Please check the above query.

Is it part of the MobileFrontend or the Vector skin?

@Jdlrobson Are they all part of Vector skin?

I referred https://github.com/wikimedia/mediawiki-extensions-examples/blob/master/tests/qunit/ext.Example.welcome.test.js and test files in MobileFrontend extension.
const

QUnit = require( 'qunit' );
Game = require ('../../modules/ext.chessViewer.core.js')

Okay Thanks.

i did the things you sad. I'll upload a patch for your reference

Thanks everyday i'm learning something new here. @Jdlrobson Sure, i need tasks that might help me get some idea about GSoC projects.

it giving
Expected:
[

"emit1",
"sub",
"foo",
"emit2",
"bar"

]
Result:
[

"emit1",
"sub",
{
  "exception": Error("Unknown module: modules/ext.chessViewer.core.js"),
  "source": "resolve"
},
"foo",
"emit2",
"bar"

]
Diff:
[

"emit1",
"sub",
{
  "exception": Error("Unknown module: modules/ext.chessViewer.core.js"),
  "source": "resolve"
},
"foo",
"emit2",
"bar"

]
error while testing it at Special:JavaScriptTest. path seems to be correct

should i also add sinon sandbox and jquery?

should i install qunit npm package as a dev dependency?

@Aklapper Thank you!!!

@Jdlrobson i added a patch.

In T321368#9999484, @Jdlrobson wrote:

i'm facing issues while committing. Is it appropriate to ask it here?

You will have more success troubleshooting in our IRC channels or Discord server https://en.wikipedia.org/wiki/Wikipedia:Discord but yes you can describe the problem and I can try and direct you to some helpful information.

In T321368#9991509, @Jdlrobson wrote:

In T321368#9991184, @Anujagrawal wrote:

@Jdlrobson What about ScrollEndEventEmitter.test.js?

Please keep but move to tests/node-qunit/mobile.special.watchlist.scripts

@Jdlrobson What about ScrollEndEventEmitter.test.js?

@Jdlrobson should i also remove the ScrollEndEventEmitter.js file from mobile.startup?

@Jdlrobson should i work on it or your patch will be merged?
i'm asking because i had set up environment for it.

Can i work on this issue and also please could someone provide more context?

Can i work on it?

@Aklapper I pinged you because you helped me in adding my first patch. But still sorry.

@Aklapper Could you add me to trusted contributors so i could sign in to gitlab?

@Soda Added a updated patch as mentioned by @Wugapodes

Thanks @Aklapper @Soda got to learn new things.
https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ChessBrowser/+/1052930

@Aklapper Check on patch failed and according to console it happened due to linting errors. What could i do?

Done @Aklapper @Soda https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ChessBrowser/+/1052930

Okay @Aklapper

@Soda Here it is bhaiya : https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ChessBrowser/+/1052930

@Aklapper Sir, i think i made a dumb move. I made changes and then submitted the patch. But when i saw diffs it showed whole code lines changed because i formatted the code in VSCode. What should i do now?

@Aklapper i added the docs string in module but i'm new using gerrit and phabricator. How should i push my changes. I asked the same on support desk but haven't been responded.

@Aklapper i cloned chessBrowser repo in extensions directory and updated localSettings.php. how could i run its development server verify the changes made?

Hey @Jdlrobson should i work on these issue?