- Issue created by @Sonal Gyanani
- Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 5:17am 1 February 2023 - 🇮🇳India rinku jacob 13 Kerala
Hi @Sonal Gyanani ,Reviewed your merge request. the README.md file was good after the merge request. But i have suggestion- title second word need to be start with capital(Component block).
- First commit to issue fork.
- Open on Drupal.org →Core: 9.5.x + Environment: PHP 8.0 & MySQL 5.7last update
over 1 year ago Not currently mergeable. - Open on Drupal.org →Core: 9.5.x + Environment: PHP 8.0 & MySQL 5.7last update
over 1 year ago Not currently mergeable. - Open on Drupal.org →Core: 9.5.x + Environment: PHP 8.0 & MySQL 5.7last update
over 1 year ago Not currently mergeable. - Open on Drupal.org →Core: 9.5.x + Environment: PHP 8.0 & MySQL 5.7last update
over 1 year ago Not currently mergeable. - 🇮🇳India ravi kant Jaipur
I have confirmed changes. README file extension is changing. But did not find any conflict in MR.
- First commit to issue fork.
- Open on Drupal.org →Core: 9.5.x + Environment: PHP 8.0 & MySQL 5.7last update
12 months ago Not currently mergeable. - last update
12 months ago 4 fail - 🇮🇳India ravi kant Jaipur
The pipline is throwing error of PHPCS and PHPCS issues is solving in another ticket https://www.drupal.org/project/component_blocks/issues/3338106 📌 Fix the errors/warnings reported by PHP_CodeSniffer Needs work .
So waiting to fix PHPCS issyes. - Status changed to Needs work
12 months ago 12:49am 5 January 2024 - 🇵🇭Philippines kenyoOwen
Hi ravi kant
I reviewed your changes in MR!6 and noticed that maintainers section is not following the README.md format for mainainter, it should be
- Daniel F. Kudwien - [sun](https://www.drupal.org/u/sun)
.
"Table of Contents" should be small letterTable of contents
Thank you.
- last update
12 months ago 4 fail - last update
12 months ago 4 fail - last update
12 months ago 4 fail - Status changed to Needs review
12 months ago 10:39am 5 January 2024 - Status changed to Needs work
12 months ago 10:16am 10 January 2024 - 🇵🇭Philippines kenyoOwen
Hi thakurnishant_06
I reviewed your changes and in the "Installation" section this text should be removed
You will need to do a cache rebuild after adding/updating layouts
since this is also stated in "Configuration" section that it needs to do run drush cr.Thank you.
- last update
12 months ago 4 fail - Status changed to Needs review
12 months ago 10:35am 10 January 2024 - Status changed to Needs work
12 months ago 5:24am 11 January 2024 - last update
12 months ago 4 fail - Status changed to Needs review
12 months ago 5:32am 11 January 2024