MR11 having merge conflicts. Please resolve that conflicts and raise PR Again
May be patch #44.Patch #44(2823847-44.patch) is worked for me. After applied patch date range view is
21 August, 2023
Used MR7 pagination is good
@alok_singh Patch is working fine. Main navigation came to right only. Please refer screen shot
Issue is closed already version 1.0.x having READme.md file.
@rviner
I have tried checking this issue. Looking like all looks good.
If i am missing anything can you please elaborate more on this issue
@zbberlin Did n't get ajax error. it is working fine after enable ajax in view. @zbberlin After enable ajax please clear cache and check.
Before updating png
After Updated png
Please refere video also
Please review and update the code with patch
working on it
kpoornima → created an issue.
@apaderno Updated MR3. Please review
looking on it
kpoornima → created an issue.
looking on it
kpoornima → created an issue.
HI @roberttabigue
MR !3 Phpcs errors checked with 8.x-2.0 and updated ticket version with 8.x-2.0.
Thanks You!
Please review and Merge
I am looking on it
kpoornima → made their first commit to this issue’s fork.
Please review and merge
looking on it
kpoornima → created an issue.
Please review
Looking on it
kpoornima → created an issue.
kpoornima → created an issue.
kpoornima → created an issue.
looking on it
kpoornima → made their first commit to this issue’s fork.
@apaderno Curious Chronicles theme enabled in drupal version 10.0.9 and Patch #3 is worked.
@apaderno Please review MR2
kpoornima → made their first commit to this issue’s fork.
Please Merge MR7 covered all files phpcs errors and warnings
kpoornima → made their first commit to this issue’s fork.
MR 8 is working.Please refer screenshot
Fixed most of the phpcs issues. Only 2 remaining warnings are:
FILE: /var/www/html/web/modules/contrib/dark_responsive-3374671/dark_responsive.theme
-------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
-------------------------------------------------------------------------------------
72 | WARNING | Unused variable $video_path.
75 | WARNING | Unused variable $video_path.
-------------------------------------------------------------------------------------
Time: 468ms; Memory: 6MB
Please Merge MR3
kpoornima → made their first commit to this issue’s fork.
@shiv_sharma Documentation in readme file it is good.
@Raveen Thakur Are you done unit testing? I wasn't able to enable this module due to dependency site module. How you can perform unit testing.
working on it
kpoornima → created an issue.
I am looking on it
Any opinions on this modules? I am waiting for the role.
kpoornima → created an issue.
shashank5563 → credited kpoornima → .
Please Merge MR 15
Subscription data is added in to user obj in MR 12. Please Review it
looking on it
kpoornima → created an issue.
kpoornima → created an issue. See original summary → .
Thanks @Shive_sharma
@shiv_sharma After testing MR 19 . I have few doubts
1)Only API Base URL field is need for connect to Nginx proxy Manger
2) With out Entering data into Email and password filed and clicked on check box of " Validate Connection" getting this message "API Base URL is Connected"
Please refer screenshot
After Merge my MR got tabs like
kpoornima → created an issue.
i am working on it
Hello folks,
I have reviewed & applied MR! 20 to my website. I am using Drupal version - 10.0.9 & PHP version - 8.1.
After Applied MR does not get error.
I am attaching my screenshots for reference.
kpoornima → created an issue.
kpoornima → created an issue.