- Issue created by @roshni27
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 12:55pm 7 July 2023 - Status changed to RTBC
over 1 year ago 11:24am 17 August 2023 Hi @roshni27, I have applied your patch.
These are the steps I followed:
1. Took clone from git version 8.x-1 in drupal 10.1.x
2. Ran this command:
./vendor/bin/phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig modules/contrib/advan/
3. Applied patch and again ran phpcs command.
found no errors.
Moving this to RTBC.
- 🇲🇩Moldova andrei.vesterli Chisinau
Hi @goodboy
Will you be able to merge the patch in the module?
Regards,
Andrei - last update
11 months ago Patch Failed to Apply - Status changed to Needs work
11 months ago 12:14pm 28 January 2024 - 🇺🇦Ukraine goodboy Kharkiv, Ukraine
@andrei.vesterli, the patch can't be applied.
I've tried to do phpcs and got a patch file with 222 rows.
So, please, reroll your patch - First commit to issue fork.
- Status changed to Needs review
11 months ago 10:16am 30 January 2024 - 🇮🇳India viren18febS
I have fixed all the issues. please review.
Thanks - last update
11 months ago 3 pass -
goodboy →
committed 984d4471 on 8.x-1.x authored by
viren18febS →
Issue #3373211 by viren18febS, roshni27: Fix the issues reported by...
-
goodboy →
committed 984d4471 on 8.x-1.x authored by
viren18febS →
- Status changed to Fixed
11 months ago 4:41pm 30 January 2024 - 🇺🇦Ukraine goodboy Kharkiv, Ukraine
@viren18febS, thanks, fixed.
Please, run test for a patch Automatically closed - issue fixed for 2 weeks with no activity.