Merge requests in need of attention - 20th November 2020

[sthursfield (Sam Thursfield)]

Recent merge requests in need of attention

Back by popular demand! (Thanks Philip

I notice that this week we have 4 MRs by the same person. They do indeed look worthy of review, but I wonder if I should limit to two per person in the list in future – what do you think?

Sound Recorder - Fix export dialog that disappears

Opened: Nov 07 (updated Nov 10)
Author: @rastersoft
URL: Fix export dialog that disappears (!164) · Merge requests · World / Vocalis · GitLab

Glade - Python plugin: Fix build against Python 3.9

Opened: Nov 09
Author: @heftig
URL: Python plugin: Fix build against Python 3.9 (!109) · Merge requests · GNOME / Glade · GitLab

Seahorse - gpgme: Fix expiry date container disabled

Opened: Nov 09
Author: @jere-ortega24
URL: gpgme: Fix expiry date container disabled (!143) · Merge requests · GNOME / Passwords and Secrets · GitLab

NetworkManager-fortisslvpn - Add 2FA capability to nm-fortisslvpn when the second factor authentication code is sent via SMS/email/etc.

Opened: Jul 26 (updated Nov 08)
Author: @emelenas
URL: Add 2FA capability to nm-fortisslvpn when the second factor authentication code is sent via SMS/email/etc. (!19) · Merge requests · GNOME / NetworkManager-fortisslvpn · GitLab

GNOME Fonts - font-view.c: Add URL to the about dialog

Opened: Nov 08
Author: @jridehalgh
URL: font-view.c: Add URL to the about dialog (!17) · Merge requests · GNOME / Fonts · GitLab

gnome-flashback - WIP: Add implementation for GF_PLACEMENT_ALIGN_ICONS_TO_GRID option

Opened: Nov 02
Author: @menschel
URL: WIP: Add implementation for GF_PLACEMENT_ALIGN_ICONS_TO_GRID option (!37) · Merge requests · GNOME / gnome-flashback · GitLab

planner - libplanner

Opened: Oct 29 (updated Oct 30)
Author: @ahmed.baizid.0
URL: libplanner API documentation (!19) · Merge requests · World / planner · GitLab

planner - Desktop Integration

Opened: Oct 29 (updated Oct 30)
Author: @ahmed.baizid.0
URL: Desktop Integration (!17) · Merge requests · World / planner · GitLab

planner - Native Language Support

Opened: Oct 29 (updated Oct 30)
Author: @ahmed.baizid.0
URL: Native Language Support (!15) · Merge requests · World / planner · GitLab

planner - POize Spanish User Guide

Opened: Oct 28 (updated Oct 30)
Author: @ahmed.baizid.0
URL: POize Spanish User Guide (!10) · Merge requests · World / planner · GitLab

[julian (Julian Hofer)]

Only two MRs per person seems like a good idea.

Not sure if I mentioned it already, but I think WIP MRs should be excluded.

[afranke (Alexandre Franke)]

FWIW these four MR have been looked at, the contributor has been reached out to, and they are working against another existing MR by a soon to be co-maintainer on planner. I think you can safely exclude these from future posts.

[sthursfield (Sam Thursfield)]

That’s great news. Shall we close the existing MRs in that case?

[p.sun.fun (Pavlo Solntsev)]

I don’t think there is a simple answer. If each MR is a simple change and a person submits N requests of simples changes (N>2) what is wrong with this? If a person submits a significant amount of MR such as proposals it is also good. They can attract brains for discussion in the earlier stage. I think this is good for the project.