Skip to content

Update app.py#370

Closed
royjor2016 wants to merge 3 commits into
toolforge:masterfrom
royjor2016:patch-6
Closed

Update app.py#370
royjor2016 wants to merge 3 commits into
toolforge:masterfrom
royjor2016:patch-6

Conversation

@royjor2016
Copy link
Copy Markdown
Contributor

Change requirement and message, so that users can apply for waiver of editcount, and are hence not incentivised to boost their editcount needlessly. https://commons.wikimedia.org/wiki/Commons:Village_pump/Proposals#c-RoyZuo-20260406210300-Video2commons_requirement_waiver

Please edit how the link is presented though. I dont know how to make it a nice looking link instead of raw url.

Change requirement and message, so that users can apply for waiver of editcount, and are hence not incentivised to boost their editcount needlessly. https://commons.wikimedia.org/wiki/Commons:Village_pump/Proposals#c-RoyZuo-20260406210300-Video2commons_requirement_waiver
no raw url?
@don-vip don-vip requested a review from Amdrel May 6, 2026 15:36
hide requirement to avoid incentivising useless edit
Comment thread video2commons/frontend/app.py
@royjor2016
Copy link
Copy Markdown
Contributor Author

I'm really noobish about code commit and stuff. I dont know what I need to do.
Please feel free to make all necessary changes and commit.

@Amdrel
Copy link
Copy Markdown
Collaborator

Amdrel commented May 15, 2026

I'm really noobish about code commit and stuff. I dont know what I need to do. Please feel free to make all necessary changes and commit.

Ah okay, no worries! I can't edit this PR due to the way GitHub works, but I can close this one and open a new one soon with the fixes applied. Thank you for your contribution!

@Amdrel
Copy link
Copy Markdown
Collaborator

Amdrel commented May 15, 2026

Alright I merged PR #379 which contains your changes along with my small edit. Thank you!

@Amdrel Amdrel closed this May 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants