- Issue created by @omkar_yewale
- @omkar_yewale opened merge request.
- Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 2:17pm 10 March 2023 - ๐ฎ๐ณIndia omkar_yewale Mumbai
!5 MR needs to be reviewed. Thank you.
- ๐ฎ๐นItaly apaderno Brescia, ๐ฎ๐น
+ $output .= '<li>' . t('Install the module "Node Save Popup".') . '</li>';Instructions on how to install the module are not much helpful, once it is installed.
+ $output .= '<li>' . t('Go to the รขโฌลConfiguration. Eg:- Admin Menu >> configuration >> services >> Node Save Configuration') . '</li>';There is no need to use an hyphen after a colon. The period is also wrong; a comma is better.
- Assigned to indrapatil
- Status changed to Needs work
almost 2 years ago 4:51pm 10 March 2023 - Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 4:55pm 10 March 2023 - ๐ฎ๐ณIndia indrapatil Bangalore
Thank you @apaderno,
I updated it according to your comment #5,
can you please review it? - Status changed to RTBC
over 1 year ago 6:00pm 20 June 2023 - ๐ฎ๐ณIndia Bushra Shaikh
I have verified and applied MR !5 on Drupal 9.5.8 version.
MR is applied successfully, and help() is added cleanly. Added screenshots for reference.Can be moved to RTBC+1.