- Issue created by @bluegeek9
- 🇺🇸United States bluegeek9
I sent an email to RoySegall → on May 25th about my desire to maintain messsage ui. I sent an email to cosolom → today about my desire to maintain message ui.
- Status changed to Needs work
over 1 year ago 4:17pm 17 June 2023 - 🇺🇦Ukraine cosolom
Hi. Can you provide some PR's or more extend description what do you want to maintain?
Giving a link to issues for that project where you posted a patch or where you reviewed patches posted from other users demonstrate you are interested in improving the project, and you are actively involved in the project. Point out the motivation behind your request. (For example, you are already using the project and you want it to be well-maintained.)
See https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or... →
- Status changed to Active
over 1 year ago 4:28pm 17 June 2023 - 🇺🇸United States bluegeek9
Hi,
My interest is making a Drupal 10 release. I am using the patch provided in 📌 Automated Drupal 10 compatibility fixes Fixed . I would be just as happy not to take over ownership; I feared the project was abandoned.
I recently took over a few abandoned projects in the quest for Drupal 10.
I am on vacation this week. I can make a few pull requests next week.
- 🇮🇹Italy apaderno Brescia, 🇮🇹
I was going to contact the project owner, since only the project owner can add new maintainers/co-maintainer, but this issue has been moved back to the project issue queue.
Hi, to my understanding cosolom is the active maintainer as I retried from Drupal stuff. I think cosolom can add co-maintainers to the project.
- 🇮🇹Italy apaderno Brescia, 🇮🇹
@RoySegall You did not give him that permission.
- 🇺🇸United States bluegeek9
Hello,
I want to be a maintainer for Message UI. I am not sure how to proceed with the maintainer (RoySegall) no longer involved with Drupal, and the co-maintainer (cosolom) cannot add maintainers. These are the task I intend to complete. My main interest is in making a stable Drupal 10 release. Below is my plan of task to complete.
Automated Tests:
The automated tests need to be updated to test when a pull request is opened.Active Pull Request:
📌 Automated Drupal 10 compatibility fixes Fixed
After completing this pull request, the automated test need to be update to Drupal 10, and php 8.1This was set as RTBC, but the patch was already committed to 8.x-1.x.
🐛 Deleted and replaced configuration entity error Closed: outdated duplicate of #3021923: Deleted and replaced configuration entity error →I reviewed this one, and hope the submitter re-submits with correction. I'll make the correction, and a pull request next week, if the submitter does not.
📌 Route "message_ui.add" does not exist. FixedI am having trouble reproducing this issue, but others have been able to reproduce it. It seems critical.
🐛 Fatal error Drupal 9 FixedI am still working on these two, but the submitted patches look promising.
🐛 JSON:API PATCH request wants permission that doesn't exist? Fixed
🐛 Langcode form element prevents translating Message entity FixedThe only other things this module could use is more automated testing, dependency injection, and some minor coding standard fixes.
📌 phpcs: coding standards Fixed - Assigned to apaderno
- Status changed to Fixed
over 1 year ago 6:43am 28 June 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
Since the project owner commented in this issue without declining the offer, and since he did not reply to the comment I posted to inform him he is the only one who can add maintainers/co-maintainers, I added bluegeek9 as maintainer (user with all the permissions on the project).
Automatically closed - issue fixed for 2 weeks with no activity.