- 🇪🇸Spain manuel.adan 🌌
Format review. Contents updated according to latest changes on code.
- Status changed to RTBC
over 1 year ago 9:02am 30 May 2023 Verified the "fixed_block_content-readme_update-3319938-5.patch" on 9.5.8 , the readme.txt file is converted into readme.md and content are updated.
- Status changed to Needs work
over 1 year ago 10:39am 30 May 2023 - 🇮🇳India Manoj Raj.R Chennai
Hi Everyone,
There is new update in the README Template.
I will work on it.
https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or... →
- last update
over 1 year ago 16 pass - Status changed to Needs review
over 1 year ago 10:46am 30 May 2023 - last update
over 1 year ago 16 pass - 🇮🇳India Manoj Raj.R Chennai
Have updated according to the latest changes from the README Template.
https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or... →Please review.
- Status changed to Needs work
about 1 year ago 7:59am 26 October 2023 - 🇮🇳India Nupur Badola
I reviewed MR!3, and the patch is not cleanly applied.
1) The text "Current Maintainers" is not required, as only current maintainers should be listed there. 2) Bulleted lists are denoted by dashes (-). 3) Add two lines prior to ##/### headings
Please follow the format of readme.md file mentioned here: https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or... →
- First commit to issue fork.
- last update
about 1 year ago 16 pass - last update
about 1 year ago 16 pass - Status changed to Needs review
about 1 year ago 9:41am 26 October 2023 - Status changed to Needs work
about 1 year ago 2:03am 27 October 2023 - 🇵🇭Philippines kenyoOwen
Hi thakurnishant_06
I reviewed your MR!3 and it's not following the README.md template. Requirements section is wrong since "HAL or Hypermedia Application Language" is Drupal core not a module needs to install before the Fixed Block Content module, the text should be "This module requires no modules outside of Drupal core.".
Thank you
- Assigned to mohd sahzad
- Issue was unassigned.
- last update
11 months ago 16 pass - Status changed to Needs review
11 months ago 5:23am 26 December 2023 - Status changed to Needs work
11 months ago 6:57am 28 December 2023 - 🇮🇳India Nupur Badola
Reviewed MR!3, the patch is not cleanly applied.
1. In Maintainers section, only one maintainer is given, so that Hyphen (-) is not required.
- Status changed to Needs review
11 months ago 9:25am 28 December 2023 - thakurnishant_06 India
@Nupur Badola, please carefully review the MR and highlight only the necessary changes according to the Drupal Readme.md template. → Regarding your comment, there is no need for this change:
In the Maintainers and Requirements section, only one maintainer/requirement is given, so the hyphen (-) is not required.
Since this section is titled "Maintainers," even when there is a single maintainer, the content of this section can be a list of maintainers, even in the case where there is just a single maintainer.
Changing back to Need Review.
- Status changed to RTBC
11 months ago 10:50pm 1 January 2024 - 🇵🇭Philippines kenyoOwen
Hi thakurnishant_06
I reviewed your changes and the issue mentioned in #14 was resolved and README.md file is following the README.md template. →
Moving to RTBC
Thank you.