🇮🇳India @bharath-kondeti

Hyderabad
Account created on 7 December 2014, about 10 years ago
#

Recent comments

🇮🇳India bharath-kondeti Hyderabad

Looks okay to me. Maintainers can take a call, if this feature is needed in this module.

🇮🇳India bharath-kondeti Hyderabad

@joachim translatable key is set, only if the language modules are enabled and setup. Maybe that's the reason its marked optional.

🇮🇳India bharath-kondeti Hyderabad

Okay. Thanks for the review. @TR do we have to address the build issues on PR in this issue itself ?

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

This looks like its working. I have tested this on Drupal 10.3 and 2.0.16 version of this module.
If you are still seeing the issue, can you please help with relevant steps to reproduce ?

🇮🇳India bharath-kondeti Hyderabad

This is not reproducible. Can we close this issue as this is raised for Drupal 8 version and sharethis module no longer supports D8 version.

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

@Grimrepear, there are two issues created and look like they are having a same context https://www.drupal.org/project/entity_share/issues/3456940 📌 Convert Entity Share plugins into attribute based plugins Active
Can you please close one, if yes ?

🇮🇳India bharath-kondeti Hyderabad

Looks like the issue is exists. It works when the hard limit box is checked, else it is allowing to type more that limited characters and its saving as well.
I have tested on Drupal 10.3.0

🇮🇳India bharath-kondeti Hyderabad

bharath-kondeti changed the visibility of the branch 3454906-d11-fixes to hidden.

🇮🇳India bharath-kondeti Hyderabad

@Rajan Kumar@2026 Can you please update the target branch in your PR from 7.x-1.x to 8.x-2.x

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

Can we use this issue for pushing all the changes cspell, phpcs, phpstan at once ?
Having 3 separate issues might lead to re-work on the individual ticket PRs again.
Can you please share your thoughts ?

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

@tim-diels Addressed cspell issues raised in this issue https://www.drupal.org/project/search_api_extras/issues/3456681 📌 Fix issues reported by CSpell Needs review . Here is the PR https://git.drupalcode.org/project/search_api_extras/-/merge_requests/4

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

@aitala
I don't see any issue post the upgrade of the module. It just enables the media library module using hook_update. It should be not due to the upgrade of this module.

🇮🇳India bharath-kondeti Hyderabad

Media Library dependency was added as it was breaking the media library add form. The fixes are tracked in this issue https://www.drupal.org/project/imagefield_tokens/issues/3282215 Support media library Fixed

I am not sure if this change can be reverted back.

🇮🇳India bharath-kondeti Hyderabad

@philiph91 This filtered results set shows up with in printed view's pdf. However, without coding the applied filter criteria cannot be shown in pdf.

Here is a workaround, try to print the filters that are added in the view using thar view's twig templates in your theme. From there, you should be able to print the filter criteria.

🇮🇳India bharath-kondeti 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 bharath-kondeti 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 bharath-kondeti 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 bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti 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 bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti 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 bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti Hyderabad

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

🇮🇳India bharath-kondeti 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.

Production build 0.71.5 2024