Account created on 14 September 2023, over 1 year ago
#

Merge Requests

More

Recent comments

🇮🇳India deepak5423

Hi @tirupati_singh,

Now it looks good to me.

Thank you.

🇮🇳India deepak5423

Hi samit,

Please solve the Merge conflicts.

Thank you.

🇮🇳India deepak5423

Thanks for the review @avpaderno,

I have updated the module accordingly.
Please review.

🇮🇳India deepak5423

Thanks for the review @avpaderno,

I have updated the module accordingly.
Please review.

🇮🇳India deepak5423

Hi,

I have tested the MR !14.
All the issues of phpstan and phpcs are fixed.
Moving the issue to RTBC.

Thanks.

🇮🇳India deepak5423

Hi,

I have updated the MR according to the review comments.
Moving the issue to Needs Review.

Thanks.

🇮🇳India deepak5423

Hi,

I have reviewed MR !12, Where the gitlab-ci file is added properly.
Moving the ticket to RTBC.

Thanks.

🇮🇳India deepak5423

Hi,

The added gilab-ci files look good to me and all the tests are also getting passed.
Moving the issue to RTBC.

Thanks.

🇮🇳India deepak5423

Hi,

The added gilab-ci files look good to me and all the tests are also getting passed.
Moving the issue to RTBC.

Thanks.

🇮🇳India deepak5423

Hi,

The added gilab-ci files look good to me and it follow the standard.
Moving the issue to RTBC.

Thanks.

🇮🇳India deepak5423

Hi,

I have reviewed MR !10, Where every tests are getting pass and gitlab ci file is added properly.
Moving the ticket to RTBC.

Thanks.

🇮🇳India deepak5423

cspell, eslint and phpstan tests are still failing.
Moving the ticket to need work.

🇮🇳India deepak5423

The gitlab-ci file is added properly and tests are also running.
Moving the ticket to RTBC.

🇮🇳India deepak5423

The file name should be .gitlab-ci.yml after doing this tests will run.
Moving the ticket to need work.

🇮🇳India deepak5423

Tests are running properly.
Moving the ticket to RTBC.

🇮🇳India deepak5423

Updated the README.md file.
Please review.

🇮🇳India deepak5423

Removed gitlab-ci file.
Moving the ticket to RTBC.

🇮🇳India deepak5423

deepak5423 made their first commit to this issue’s fork.

🇮🇳India deepak5423

cspell tests are now passing.
Moving the ticket to RTBC.

🇮🇳India deepak5423

PHP unit test is now running and all PHP unit tests are passing.
Moving the ticket to RTBC.

🇮🇳India deepak5423

All the pipelines are passing.
Moving the ticket to RTBC.

🇮🇳India deepak5423

Hi @alexpott,

Phpunit test is still failing.
Moving the ticket to RTBC.

🇮🇳India deepak5423

Gitlab ci file added and all tests are passing.
Moving the ticket to RTBC.

🇮🇳India deepak5423

eslint test are passing.
Moving the ticket to RTBC.

🇮🇳India deepak5423

cspell test is passing.
Moving the ticket to RTBC.

🇮🇳India deepak5423

The pipeline is running properly and gitlab ci file is added properly.
Moving the ticket to RTBC.

🇮🇳India deepak5423

Hi dtfabio,
Stylelint tests are now passing.
Moving the ticket to RTBC.

🇮🇳India deepak5423

Hi idebr,
There is no error left for eslint so moving the ticket to RTBC.

🇮🇳India deepak5423

Hi heddn,
eslint is still failing so moving the ticket to need work.

🇮🇳India deepak5423

Gitlab-ci template is added properly. Tests are also running.
Moving the ticket to RTBC.

🇮🇳India deepak5423

@nilesh.addweb phpstan test is still failing.
Moving the ticket to NW.

🇮🇳India deepak5423

Gitlab-ci file is properly add and tests and pipeline is running.
Moving the ticket to RTBC.

🇮🇳India deepak5423

Gitllab ci file is properly added.
Moving the ticket to RTBC.

🇮🇳India deepak5423

Gitlab ci added properly.
Moving the tickek to RTBC.

🇮🇳India deepak5423

@eiriksm The changes looks fine drupalci file is deleted in MR !11.
Moving the ticket to RTBC.

🇮🇳India deepak5423

I have checked the issue and now its pass all the gitlab pipeline tests.
Moving the ticket to RTBC.

🇮🇳India deepak5423

Hi, @markie thanks for the update and for closing this issue. However, I noticed no credit given to me for resolving this issue. Is there something I am missing! :)

🇮🇳India deepak5423

Hi,
I have noticed the changes mentioned earlier and have fixed them.

Regarding comment #14, point 6: Since we wanted to have a popup on the same screen without any redirection to another route for confirm form, we decided to stick with the JavaScript popup. As for the bulk operation link to delete the paragraph types, it seems like a functionality we might consider implementing in a future release of the module if needed.

Moving to Needs Review.
Please review.

Thanks.

🇮🇳India deepak5423

Hi @stephen.geinosky,
Yes, tests should be in a separate ticket.

🇮🇳India deepak5423

Hi @stephen.geinosky,
There was a small mistake in the file name convention, I updated it.
Please review.

🇮🇳India deepak5423

deepak5423 made their first commit to this issue’s fork.

🇮🇳India deepak5423

deepak5423 made their first commit to this issue’s fork.

🇮🇳India deepak5423

Hi @riddhi.addweb,
Please try to apply patch on 2.0.x, and if its still not works please let me know

🇮🇳India deepak5423

I have pulled the latest changes of 2.0.x branch and Fixed the phpcs error.
Please review.

🇮🇳India deepak5423

phpunit test looks like something unexpected could someone else look into this

🇮🇳India deepak5423

Thanks for the reviews.

I have added the changes accordingly.
Please review.

🇮🇳India deepak5423

I have made the changes. The project is now ready to be reviewed.

🇮🇳India deepak5423

@webfaqtory I think it is better to add file_delete module as a dependency of the File Replace (D8+)

🇮🇳India deepak5423

deepak5423 changed the visibility of the branch 1.x to hidden.

🇮🇳India deepak5423

deepak5423 changed the visibility of the branch 1.x to hidden.

🇮🇳India deepak5423

deepak5423 made their first commit to this issue’s fork.

🇮🇳India deepak5423

deepak5423 made their first commit to this issue’s fork.

🇮🇳India deepak5423

deepak5423 made their first commit to this issue’s fork.

🇮🇳India deepak5423

deepak5423 changed the visibility of the branch 8.x-1.x to hidden.

🇮🇳India deepak5423

deepak5423 changed the visibility of the branch 1.0.x to hidden.

🇮🇳India deepak5423

deepak5423 changed the visibility of the branch 3428012-1 to hidden.

🇮🇳India deepak5423

deepak5423 made their first commit to this issue’s fork.

🇮🇳India deepak5423

@Vishal Prasad !MR55 is not following the README.md standards. Please follow the README.md standards.

🇮🇳India deepak5423

@kalash-j there are still some phpcs issue left.

🇮🇳India deepak5423

I have applied !MR7 no phpcs found.
It looks good to me.

🇮🇳India deepak5423

After applying !MR5 no phpcs error was found.
Looks good to me.

Production build 0.71.5 2024