🇮🇳India @kbk1992

Hyderabad
Account created on 7 December 2014, over 9 years ago
#

Recent comments

🇮🇳India kbk1992 Hyderabad

Is this issue on Drupal 11 or Drupal 10 ?
This module does not have a release for Drupal 11 yet. It must be throwing from a patch from this issue https://www.drupal.org/project/imageapi_optimize_webp/issues/3448709 📌 Automated Drupal 11 compatibility fixes for imageapi_optimize_webp Needs review

🇮🇳India kbk1992 Hyderabad

I think there is no new field type or field widget as per the module feature. With respect to the steps given in README, I think this module works as designed.

🇮🇳India kbk1992 Hyderabad

Can you please help with Drupal versions and module versions where you are seeing this issue ?
Add steps to reproduce as well, if possible

As of now, with Drupal 10.2.7 I can see the block is getting placed and don't see any issue there.

🇮🇳India kbk1992 Hyderabad

Somehow not able to reproduce this issue, Can you please help with testing steps where you are seeing this issue ?

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

The patch file is already fixed in this issue https://www.drupal.org/project/legal/issues/2897486#comment-15572202 🐛 Don't log out users who do not accept the T&C Needs review .

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

Reverted the blank line code on Syndicati‎onItem.php for failing tests. I will working on the remaining.

🇮🇳India kbk1992 Hyderabad

@MegaChriz Tried to fix the review. Please check and do let me know, if anymore changes are needed.

🇮🇳India kbk1992 Hyderabad

This is not reproducing with the given steps on the Drupal 10.2.6 version.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

@tssarun Not able to reproduce this issue with same configurations. Can you please help with more specific steps or use-case where you are facing this issue ?

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

Addressed review and moved changes to a PR. Moving the issue to Needs Review

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

Hey @axelm and everyone, any update on the upgrade and can someone help with any workaround. I tried with the pulling the patch from the PR. Its not working.

🇮🇳India kbk1992 Hyderabad

Can we get a D10 release for this module, please ?

🇮🇳India kbk1992 Hyderabad

Attaching the screenshot for reference, its not working for anonymous users.

🇮🇳India kbk1992 Hyderabad

Re-opening this issue. This is still not working on D10 version of this module

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

@apaderno
From the issue summary, we have 3 files with end on line issues "End of line character is invalid; expected "\n" but found "\r\n""
Git ignores this while pushing the changes. I think that is the reason 3 files are missing from the patch..

🇮🇳India kbk1992 Hyderabad

Raised a new PR with few phpcs fixes.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

The MR has been raised with only the changes are not present in the codebase when compared with the patch.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

🇮🇳India kbk1992 Hyderabad

Addressed #15 and raised a MR. Please review.

🇮🇳India kbk1992 Hyderabad

bharath-kondeti made their first commit to this issue’s fork.

Production build 0.69.0 2024