Account created on 4 June 2024, 10 months ago
#

Merge Requests

More

Recent comments

🇮🇳India sayan_k_dutta

Reviewed the MR. The changes seem to resolve the issue. All the custom blocks created are now visible under blocks options in gutenberg editor. Attaching screenshots for the same and moving this to RTBC.

🇮🇳India sayan_k_dutta

Applied the MR and it seems to resolve the issue. Attaching screencasts for reference and moving it to RTBC.

🇮🇳India sayan_k_dutta

Cloned the repo and reviewed the MR. It seems that the MR solves the issue of PHP Fatal Error perfectly which was occuring during enabling the module. Attaching screencasts for reference and moving this to RTBC.

🇮🇳India sayan_k_dutta

Cloned the repo and checked the MR. Before applying the MR, when checking for upgrade status, under deprecated api use, the error "Twig Filter "spaceless" is deprecated" was listed. Applied the MR and it seems to fix the issue as no "spaceless" decrecation error is listed now for the upgrade status of Seven theme.
The MR looks good to me and moving this to RTBC.

🇮🇳India sayan_k_dutta

Installed the Provus distribution with Drupal 10.3.6. Checked the MR and verified that the title and body of the accordian sections are made required. Attaching a screencast for reference. Moving this to RTBC.

🇮🇳India sayan_k_dutta

Reviewed the MR in Drupal Core 10.4.5. Steps followed for testing the MR.

  1. Cloned the repo and checked out the branch 3361765-footer-menus-overlapping.
  2. Installed the Kite theme from Appearance section.
  3. Under Structure>Menus created 4 menus named Footer 1, Footer 2, Footer 3, Footer 4 in addition to the default footer menu present.
  4. Added links under all the five menus.
  5. Placed the menus under Footer Top, Footer First, Footer Second, Footer Third, Footer Fourth, with each menu in each footer.
  6. Observed the changes in the footer region after creating any node page.

The changes looks good to me. Moving this to RTBC. Attaching screenshot for reference:

🇮🇳India sayan_k_dutta

@mrdalesmith, works perfectly and passing all tests. You can move it to RTBC.

🇮🇳India sayan_k_dutta

The patch in #11 solves the problem. Both underscores and hyphens are now applied to menu machine names. By default, as a replacement underscores are applied, inline with the naming convention of other machine names. Created MR !11592 for the same.
The screenshots are attached below after testing on my local:




Need others to review.

🇮🇳India sayan_k_dutta

Made the changes. Removed the 'fieldgroup' from #type and added it as a class attribute.

🇮🇳India sayan_k_dutta

Made the changes. Please review MR !218

🇮🇳India sayan_k_dutta

Working further to convert the views_hooks in oop implementation.

🇮🇳India sayan_k_dutta

sayan_k_dutta changed the visibility of the branch 3489482-switch-to-oop to active.

🇮🇳India sayan_k_dutta

sayan_k_dutta changed the visibility of the branch 3489482-switch-to-oop to hidden.

🇮🇳India sayan_k_dutta

Please review the changes. Then we can add the new parameter if all the hooks have been changed.

🇮🇳India sayan_k_dutta

Working on the further changes.

🇮🇳India sayan_k_dutta

Installed drupal on the branch drupal-3502094/'3502094-when-saving-the with the minimal profile.
Navigated to 'Role Settings' options and after confirming that the admin role was not created by default, created a new role named Administrator. The added the role in the Role Settings- confirmed that the status message 'The role settings have been updated.' is shown when clicked on Save Configuration button.
Attaching a screencast for the same and moving it to RTBC.

🇮🇳India sayan_k_dutta

Added the tests. Please review.

🇮🇳India sayan_k_dutta

Added the hook classes for the main modules. Adding for submodules as well.

🇮🇳India sayan_k_dutta

Tested the MR. The key integration with the Graph mail module seems to work fine.
Created a key of type Graph Mail and configured the graph api mail settings to work to with the key created. Used dummy values for testing. All the comments on the MR also seems to be addressed. Attaching a video for reference and moving it to RTBC.

🇮🇳India sayan_k_dutta

Tested the MR and configured the conditional fields with select2 widget. The select2 field widget is now working fine showing only the field whose dependency is added related to the value in list field. Attaching screencasts for the same and moving it to RTBC.

🇮🇳India sayan_k_dutta

I checked the change password page. It is not accessible and the page throws error if the password policy module is enabled.
The MR fixes this and the page is accessible now. Moving this to RTBC.

🇮🇳India sayan_k_dutta

Used drush en fullcalendar to install the module on the branch 3497050-warning-message-on. The warning message Route "admin/config/user-interface/fullcalendar" does not exist. does not appear anymore.
There is a phpstan error on pipeline but it does not seem to be related to code changes made for this issue. Hence moving it to RTBC.

🇮🇳India sayan_k_dutta

Created MR 20 in-cooperating changes from the 3.0.x branch. Please review this MR.

🇮🇳India sayan_k_dutta

Phpunit tests are still failing for some reason. If anyone can look into it.

🇮🇳India sayan_k_dutta

Checking using FieldableEntityInterface seems to filter out the entity types without any fields from the source entities list. Created the MR !116 according to it.
I have not tried using the EntityUsageTrackBase as I am not clear of the idea of how to use it to check for entities without fields.

🇮🇳India sayan_k_dutta

I was able to delete the custom tokens without any error. Can you provide the steps to reproduce the issue.

🇮🇳India sayan_k_dutta

Made the changes requested. Please review.

🇮🇳India sayan_k_dutta

Made the code changes according to it, following @dom 's suggestions and reopened the MR !9.
I have reviewd from my end and it is working fine. Need others to confirm the review.

🇮🇳India sayan_k_dutta

@dom.
I applied your patch and used the views bulk edit module with simplify. Seems to resolve the errors previously occuring. Thanks for the help.
Attaching the screencasts.

🇮🇳India sayan_k_dutta

The pipeline is still failing for missing return types in the following files

core/lib/Drupal/Component/Annotation/Reflection/MockFileFinder.php
core/lib/Drupal/Component/Diff/DiffFormatter.php
core/lib/Drupal/Core/Diff/DiffFormatter.php

It has also some spell check errors.
Moving it to needs work. The pipeline needs to be fixed. Help from someone will be welcome.

🇮🇳India sayan_k_dutta

Made the required changes. Please review MR !704.

🇮🇳India sayan_k_dutta

All tests are now passing, warnings are resolved. Please review.

🇮🇳India sayan_k_dutta

I have made the code changes to remove the empty rows whenever an entity is saved in the database. Also the empty rows are removed from being rendered on the webpage.
Please review MR !33.

🇮🇳India sayan_k_dutta

Tested the changes in the MR 8. The dropdown menu is visible after the code changes.. Attaching screencasts before and after applying the patch for reference and moving it to RTBC.

🇮🇳India sayan_k_dutta

Tested the MR and applied the theme. The changes seem to be fine and all the menu levels are now visible. Attaching a screenshot for reference and moving it to RTBC.

🇮🇳India sayan_k_dutta

Reviewed the MR. The issue seems to be resolved if the image_autorotate plugin is removed.
Attaching a screencast for the same and moving to RTBC.

🇮🇳India sayan_k_dutta

Checked again, works fine for me @koustav_mondal. Please check the branch once and the specific drupal version for it.

🇮🇳India sayan_k_dutta

Made the changes. Please review.
Also the pipeline was failing as the tests module was incompatible with D11. Made the fix for it. Pipeline is green now.

🇮🇳India sayan_k_dutta

Reviewed the MR. Checked all the changes made.
1. Checked that on submitting more than one empty components for the name field, all the "_none" values are replaced by empty strings.
2. Checked that the validation does not trigger if we give "_blank" or "_none" as input.
3. Checked all the tests written and all of them passed.

Hence moving it to RTBC.

🇮🇳India sayan_k_dutta

Thanks @jdewit, overlooked it. Made the changes.

🇮🇳India sayan_k_dutta

Made the changes and added tests. Please review.

🇮🇳India sayan_k_dutta

Made the requested changes. Please review.

🇮🇳India sayan_k_dutta

Made the changes. Please review.

🇮🇳India sayan_k_dutta

Made the code changes required to display a option to enter lang attribute in the link form. But drupal ckeditor5 seems to filter out the attribute, as it is not being rendered in the link form. Someone please review the code changes.

🇮🇳India sayan_k_dutta

Made the changes. Verified working fine.
Please review.

🇮🇳India sayan_k_dutta

Made the required changes, please review.
@niharika.s please make the MR 59 as ready.

🇮🇳India sayan_k_dutta

sayan_k_dutta changed the visibility of the branch 3498303-php-8.4-compatibility to active.

🇮🇳India sayan_k_dutta

I have made the changes by removing the depricated options, please review.

🇮🇳India sayan_k_dutta

Assigned mistakenly. But I want to highlight a point that adding the content type column is causing the table cell for the content upload time to shrink and it is not looking good in appearance. Maybe we can do something to restore the widths of the column.

🇮🇳India sayan_k_dutta

Made the changes as requested. Please review.

🇮🇳India sayan_k_dutta

@zenimagine, I have created MR !24 with the changes. Please check if it works.

🇮🇳India sayan_k_dutta

We then have to modify the .module file to make the necessary changes. I am looking into it.

🇮🇳India sayan_k_dutta

@prem suthar, thanks for the follow up, then I think the module works as designed.

🇮🇳India sayan_k_dutta

Some specific case arise:

  1. User registers for a new account but is pending approval from admin.
  2. User registers for a new account but has not yet validated their email.
  3. User registers for a new account but both admin approval and email validation is pending.

In these cases the user is still not logged in. So just after creating the account, will they be redirected to the set page?
Need others' thoughts.

🇮🇳India sayan_k_dutta

The issue arises when the new account created needs to be validated through email or by admin. Seems like the redirection checking is bypassed if the user is not logged in. I am using version 2.x-dev@dev on drupal 10.3.10.

🇮🇳India sayan_k_dutta

Seems like RestrictIpAcessTest.php is not correctly executing the test for Blacklisted paths. I don't have much knowledge about unit tests. Hence unassigning myself. Anyone please look into it to move it forward.

🇮🇳India sayan_k_dutta

@maintainers, enabling the deprecated attributes is causing the ckeditor to function incorrectly. If for end users of the module it is meant not to use or enable those attributes, can those be removed from the config form?

🇮🇳India sayan_k_dutta

Again tried to reproduce the error in Drupal 10.4.0, there seems to be no issue for 2.x branch of the modulew. On first time Tfa setup with TOTP, in Tfa configuration page if the "Redirect users on login to TFA Setup Page" option is enabled, the user is correctly redirected to the tfa overview form asking the user to set up tfa.

🇮🇳India sayan_k_dutta

Reviewed the changes. Looks good. Attaching screenshot of the same and moving to RTBC.

🇮🇳India sayan_k_dutta

@igorgoncalves yes, indeed the two strings of urls are different, that is what we need here. Do I need to make changes to test files, to remove the warnings. I don't have much knowledge of phpunit tests, but can look into it.

🇮🇳India sayan_k_dutta

Reviewed the changes. Looks Good. Attaching screenshot of the same and moving to RTBC.

Production build 0.71.5 2024