๐Ÿ‡ฎ๐Ÿ‡ณIndia @Jitendra verma

Delhi
Account created on 23 September 2011, almost 13 years ago
#

Recent comments

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

@cilefen Thanks for your reply.
As in the contact module used the implode method, when fresh from load it gives an array, but when form rebuilds it gets data from the direct field which is strig that why my Ajax call was broken, if we can add there a check for array it will resolve Issue .

if there is any other solution that I can use to solve my problem, Please help

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

Thanks @apaderno, I created a separate issue please have a look https://www.drupal.org/project/adminer/issues/3450524 ๐Ÿ’ฌ Offering to maintain Adminer Active .

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

Hi @apaderno
I would like to update this module. Could you please add me as a maintainer?

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

Sure @astonvictor, Please update your code , I mentioned in the Issue thread.and closing this thread.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

Jitendra verma โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

Hi there,

I'd like to work on the Drupal 8,9,10 version to create a stable release. So, I want to propose to add me as a co-maintainer.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

@BramDriesen sure i will merge request move it in to separate issue,

As i shown interest in become project maintainer before 28 days , and i asked alll maintainers of this project but they don't have permissions to add me as maintainer , Please Help

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

I fixed the current Issue and cread MR , Please add me as Maintaner , so i can update module reguallarly.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

I want to update this module , Please add me as maintainer.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

Thank you for applying! Reviewers will review the project files, describing what needs to be changed.

Please read Review process for security advisory coverage: What to expect โ†’ for more details and Security advisory coverage application checklist โ†’ to understand what reviewers look for. Tips for ensuring a smooth review โ†’ gives some hints for a smother review.

To reviewers: Please read How to review security advisory coverage applications โ†’ , What to cover in an application review โ†’ , and Drupal.org security advisory coverage application workflow โ†’ .

Since the project is being used for this application, for the time this application is open, only the user who created the application can commit code.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

Please fix PHPCS issues
phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml web/modules/contrib/taxonomy_scheduler

FILE: /app/web/modules/contrib/taxonomy_scheduler/README.md
----------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
----------------------------------------------------------------------
52 | WARNING | Line exceeds 80 characters; contains 93 characters
59 | WARNING | Line exceeds 80 characters; contains 82 characters
----------------------------------------------------------------------

FILE: ...cheduler/src/EventSubscriber/TaxonomySchedulerCronSubscriber.php
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
16 | WARNING | The class short comment should describe what the
| | class does and not simply repeat the class name
----------------------------------------------------------------------

FILE: ...duler/src/EventSubscriber/TaxonomySchedulerPresaveSubscriber.php
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
16 | WARNING | The class short comment should describe what the
| | class does and not simply repeat the class name
----------------------------------------------------------------------

FILE: .../taxonomy_scheduler/src/Exception/TaxonomySchedulerException.php
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
7 | WARNING | The class short comment should describe what the class
| | does and not simply repeat the class name
----------------------------------------------------------------------

FILE: ...pp/web/modules/contrib/taxonomy_scheduler/src/Form/AdminForm.php
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
17 | WARNING | The class short comment should describe what the
| | class does and not simply repeat the class name
----------------------------------------------------------------------

FILE: ...taxonomy_scheduler/src/Service/TaxonomySchedulerFieldManager.php
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
16 | WARNING | The class short comment should describe what the
| | class does and not simply repeat the class name
----------------------------------------------------------------------

FILE: .../taxonomy_scheduler/src/ValueObject/TaxonomyFieldStorageItem.php
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
9 | WARNING | The class short comment should describe what the class
| | does and not simply repeat the class name
----------------------------------------------------------------------

FILE: ...axonomy_scheduler/src/ValueObject/TaxonomySchedulerQueueItem.php
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
9 | WARNING | The class short comment should describe what the class
| | does and not simply repeat the class name
----------------------------------------------------------------------

FILE: ...contrib/taxonomy_scheduler/tests/src/Kernel/FieldStorageTest.php
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
13 | WARNING | The class short comment should describe what the
| | class does and not simply repeat the class name
----------------------------------------------------------------------

FILE: ...xonomy_scheduler/tests/src/Unit/TaxonomyFieldStorageItemTest.php
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
11 | WARNING | The class short comment should describe what the
| | class does and not simply repeat the class name
----------------------------------------------------------------------

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

Please change the title of issue in this format:
[latest branch] Module name

So that reviewers can check your code.

Thanks

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

Thank you for applying! Reviewers will review the project files, describing what needs to be changed.

Please read Review process for security advisory coverage: What to expect โ†’ for more details and Security advisory coverage application checklist โ†’ to understand what reviewers look for. Tips for ensuring a smooth review โ†’ gives some hints for a smother review.

To reviewers: Please read How to review security advisory coverage applications โ†’ , What to cover in an application review โ†’ , and Drupal.org security advisory coverage application workflow โ†’ .

Since the project is being used for this application, for the time this application is open, only the user who created the application can commit code.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

@apaderno
All the code review comments have been addressed and committed into the branch - 2.0.x . Please review.

Git repo URL - https://git.drupalcode.org/project/social_feeds_block/-/tree/2.0.x
Branch - 2.0.x

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

Thank you for the feedback.

All the code review comments have been addressed and committed into the branch - 2.0.x . Please validate the same and let me know please if there is any observation.

Git repo URL - https://git.drupalcode.org/project/social_feeds_block/-/tree/2.0.x
Branch - 2.0.x

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

Hi Apaderno,

I crated a sperate post for another Module, Please review https://www.drupal.org/project/projectapplications/issues/3336321 โ†’

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

Hi Apaderno,

Thanks , Yes i am working on another module aswell , I will share that module with you soon.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jitendra verma Delhi

Hi vishal.kadam & jitesh_1,

Thanks for review,As suggested I have Replace README.txt with README.md and fixed PHPCS Issues https://git.drupalcode.org/project/contact_mail_formatter/-/commit/fd6af...

Production build 0.69.0 2024