Problem/Motivation

The list of the module maintainers are outdated.

Proposed resolution

Add @paulocs, @mlsamuelson @a.dmitriiev to the list.

CommentFileSizeAuthor
#8 insert_view-Update_maintainers_in_README_txt_file-3182756-d8-8.patch566 byteskleiton_rodrigues
#3 3182756-applied_patch.png62.32 KBabhijith s
#2 insert_view-Update_maintainers_in_README_txt_file-3182756-d8-2.patch566 bytespaulocs

Comments

paulocs created an issue. See original summary.

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Assigned: paulocs » Unassigned
Status: Active » Needs review
StatusFileSize
new insert_view-Update_maintainers_in_README_txt_file-3182756-d8-2.patch566 bytes

Follow a patch.

abhijith s’s picture

abhijith s commented
StatusFileSize
new 3182756-applied_patch.png62.32 KB

Applied patch #2 and it works fine. The maintainers name and profile Urls are added correctly in patch.

after

abhijith s’s picture

abhijith s commented
Status: Needs review » Reviewed & tested by the community
renatog’s picture

renatog commented

Really makes sense. Its necessary to keep it update.

+1

renatog’s picture

renatog commented
Status: Reviewed & tested by the community » Needs work

Just one minor item on this part:

* Paulo Henrique Starling (paulocs) - https://www.drupal.org/u/paulocs
+ * Michael Samuelson (mlsamuelson) - https://www.drupal.org/u/mlsamuelson
+ * Artem Dmitriiev (a.dmitriiev) - https://www.drupal.org/u/admitriiev

It is better to use ID stead of username.

Example: Stead of https://www.drupal.org/u/paulocs its recommended to use: https://www.drupal.org/user/3640109 and so on

This recommendation is because "username" can be changed but user_id not. So if someone changes the username the user_id will aways be the same and it will keep the reference

Makes sense?

kleiton_rodrigues’s picture

kleiton_rodrigues commented
Assigned: Unassigned » kleiton_rodrigues

I'm work

kleiton_rodrigues’s picture

kleiton_rodrigues commented
Assigned: kleiton_rodrigues » Unassigned
Status: Needs work » Needs review
StatusFileSize
new insert_view-Update_maintainers_in_README_txt_file-3182756-d8-8.patch566 bytes

Follow a patch.

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Status: Needs review » Reviewed & tested by the community

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Status: Reviewed & tested by the community » Fixed

Fixed in dev branch.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.