Added the hook classes for the main modules. Adding for submodules as well.
Working on it.
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.
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.
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.
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.
Created MR 20 in-cooperating changes from the 3.0.x branch. Please review this MR.
Phpunit tests are still failing for some reason. If anyone can look into it.
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.
Working on it.
I was able to delete the custom tokens without any error. Can you provide the steps to reproduce the issue.
Made the changes requested. Please review.
Made the changes. Please review.
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.
@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.
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.phpIt has also some spell check errors.
Moving it to needs work. The pipeline needs to be fixed. Help from someone will be welcome.
Working on it.
Working on it.
Made the required changes. Please review MR !704.
Working on it.
Made the requested change. Please review.
All tests are now passing, warnings are resolved. Please review.
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.
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.
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.
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.
Checked again, works fine for me @koustav_mondal. Please check the branch once and the specific drupal version for it.
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.
Working on it.
Made the change. Please review.
Working on it.
Working on it.
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.
Thanks @jdewit, overlooked it. Made the changes.
Here is the patch attached.
Made the changes and added tests. Please review.
Made the requested changes. Please review.
Working on it.
Ok @dcam, unassigning myself.
Working on it.
@yas changes done.
Made the changes. Please review.
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.
Made the changes requested.
Made the changes. Verified working fine.
Please review.
Working on it.
Working on it.
Made the required changes, please review.
@niharika.s please make the MR 59 as ready.
sayan_k_dutta → changed the visibility of the branch 3498303-php-8.4-compatibility to active.
I have made the changes by removing the depricated options, please review.
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.
Working on it.
Made the changes as requested. Please review.
Working on it.
Working on it.
@zenimagine, I have created MR !24 with the changes. Please check if it works.
We then have to modify the .module file to make the necessary changes. I am looking into it.
@prem suthar, thanks for the follow up, then I think the module works as designed.
Some specific case arise:
- User registers for a new account but is pending approval from admin.
- User registers for a new account but has not yet validated their email.
- 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.
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.
Working on it.
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.
@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?
Working on it.
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.
Reviewed the changes. Looks good. Attaching screenshot of the same and moving to RTBC.
@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.
Reviewed the changes. Looks Good. Attaching screenshot of the same and moving to RTBC.
Looking into it.
I don't have the required knowledge for the failing tests. Please someone look into it.
Unassigning myself.
Unassigning myself as it is a duplicate issue and has been closed.
Working on it.
Made the requested changes.
Tested the changes. Working as expected. Attaching a screenshot.
Ok looking into it.
Working on it.
Made the changes on MR !20. Please review.
Working on it.
Working on it.
Made the changes. Please review.
Working on it.
@thomas.frobieter should I open a MR making the required changes?
Created the MR !168 with the necessary changes. Please review.
Working on it.
The label field already shows the set label for the tag. Is it correct that the name field will also show the same.
Working on it.
Created the MR !7 making the required changes.
Please review.
Created the MR !9.
Working on it.
No more phpstan errors.
Checking if it solves the issue.