@kushagra.goyal

kushagra.goyal → created an issue.

Hi, as i mentioned in last comment, not able to see any errors in drupal logs. When we are hitting 'Save' button (blue icon from drupal). there is one API called which is https://uat.startupindia.gov.in/sih/dxpr_builder/ajaxtoken=rywoK8LvktEiE... which comes under 'dxpr_save_container' AJAX request call.

but this gives us '403 Forbidden' as a response. previously i was thinking it was a permission issue because i was using free license of dxpr. but i notice that it sometimes working but mostly didn't. so i have also tried with purchased license but didn't get success on it.

Hi, we are not getting any error in Drupal log, it think it is because this issue not from Drupal side, we just got red warning error and when we tried to save dxpr changes by licking on blue save icon and in network tab an api getting failed (image attached). so there will be error somewhere else...

The thing is we are not getting any error in local. this reproducible on live (uat, dev) sites.

kushagra.goyal → created an issue.

Applied the MR changes on local, it is working fine and changes are working fine and now Pagination is now looking good. Attached before and after images for reference. Hence RTBC+1..

Issue reproduced and applied MR !6 changes on local for missing php tag, changes applied successfully and works fine now. Hence RTBC+1..

Applied MR !57 changes on local, it fix the issue successfully and working fine. Now there is no space is there before before full stop, comma. Attached Before and After image for reference. Hence +1 RTBC..

Hi @elber, Thanks for patch. I have applied the patch and tested on local it is resolving the issue. Hence +1 RTBC...

I have verified that MR !50 resolving the issue of image is not being uploaded which is grater than 15MB, No other issue observed Adding Before and After images for reference. Hence +1 RTBC.

And @sundarraj.p you can take reference of this documentation to test fixes - https://www.drupal.org/docs/develop/git/using-gitlab-to-contribute-to-dr... →

Hi @yustinTR for patch, #30 patch looks good to me, applied successfully and test changes on local it is now compatible and working fine for me. Hence green flag to RTBC+1...

#8 patch is not applying because the file name is different i have checked the branch as well so updating the patch along with interdiff, working fine and it resolve the issue..

Reviewed on this issue and reproduced. After applying the patch error got resolved successfully. Attached before and after screenshots. Good to move RTBC..

After applied the patch issue got fixed and now successfully compatible with D10. Attached before and after screenshot for reference. Hence RTBC+1..

Updating patch again as above mentioned patch in #13 have some issues.

After removing the description there is no longer necessary to write tests for description and test-cases as encountered failures in patch #11, So according updating the Patch to resolve failure.
Thanks..

Reviewed and Applied changes of #4 in local. It creates Schedule maintenance disable field, Attached Before and After screenshots for reference. RTBC+1..

In #2 there is a test-case failure So, Adding a updated patch for fixing failure test-cases. Attached the patch plz review.

Patch #3 is applying correctly and it solves the offset error but i observed that Changing the value of the "slack app" field is not triggering update Ajax of the form, So accordingly i have update a event and verified it is working fine now.
Please review..

Verified the latest patch and after applying #4 patch it is working fine. For reference attaching before and after screenshots.

i have reviewed the patch it's working correctly as before it was typo error. and i have checked previously it was giving null but now it is working fine. Hence moving to RTBC..

I have looked to this issue, MR!375 is resolving the issue as replacePlain do not contain Markup Involvement so, it doesn't convert special char to html markups, hence works as expected, Adding before after screenshots for reference..

As per the comment #7 there was some merge conflicts in MR!2 so for resolve that issue i have raised new MR!4. Hence moving to needs review..

kushagra.goyal → made their first commit to this issue’s fork.

I reviewed the issue and this is reproduced for me. After applying the patch issue got resolved. and it's working successfully fine, for additional here i am attaching Screenshots and moving this to RTBC.
Thanks...

kushagra.goyal → created an issue.

I review the patch associated with this issue, and it is working flawlessly. after applying patch #2 module is not now compatible with Drupal 10, Attached screenshot after applied a patch. Hence moving to ahead as RTBC

Hi, Patch #2 was not getting applied correctly, created a MR #7 which resolves error. Please review..

kushagra.goyal → made their first commit to this issue’s fork.

Hi @markdorison, i have reviewed the MR at #2. Verified all the changes attached in MR. working fine no issue has been observed on this is working as expected. Hence moving to RTBC...

MR !13 is applying correctly. no other issues not observed as it is working as expected. module is compatible with D10 Hence moving to RTBC. screenshot Attached..

Reviewing this issue...

Patch created in #2, Resolving issue and works as expected, Needs more eyes to view on that, Moving to Needs Reviews.

kushagra.goyal → created an issue.

Hi @rajan, this works. Thanks for really quick response. Really appreciated..

kushagra.goyal → created an issue.

#3 working perfectly for me instead of applying patch, thanks...

Thank you so much @salmonek for addressing this bug so quick. Your quick response is greatly appreciated...

kushagra.goyal → created an issue.

Last #21 was having some git error, updating with the new patch.

re updating the patch and and addressing CCF also attaching inter_diff along with it.

applying a patch to resolve all the PHP coin

kushagra.goyal → created an issue.

There was a lot of messed up at above and i made little alterations in the patch path, so updating patch along with interdiff file.

Updating #17 interdiff file.

Hi @smustgrave, there is a lot of changes on #15 and i verified by applied #11 patch but on that any of the tokens are not visible in available tokens list. that patch #11 is applying but not working. As i've not mention interdiff file, so attaching now...

Applying the patch for the the current issue, its work alright, Please review.