🇮🇳India @ankitv18

Looks fine!!

ankitv18 → created an issue.

ankitv18 → created an issue.

Looks good

Attempted to create a logo
If looks good then we can push into the MR

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

I’m not seeing any changes in .info.yml
Also we can remove D7 reference from readme.md file
As we should support only D10 and above

Provides a simple, standalone machine-name field.
Use this if you only need a single machine-name input.
For two linked fields (a common Drupal core pattern), choose Safeword instead.

How’s this sound?

MR merged hence moving into fixed state.

MR!41 is ready for a review.

Hey @marcus
It seems you have missed to give the credit for this fix.

MR!191 is ready for a review.

ankitv18 → created an issue.

Moving into review.

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

ankitv18 → created an issue.

MR!40 is ready for a review.

ankitv18 → changed the visibility of the branch 3111456-language-issues to hidden.

Closing in favour of https://www.drupal.org/project/acquia_optimize/issues/3544753 📌 Rename to Acquia Web Governance Active

ankitv18 → created an issue.

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

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

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

ankitv18 → created an issue.

Looks good to me, hence moving this to LGTM

MR!39 is ready for a review.

Hi @anybody @danrod
Except Phpcs(which should be covered in https://www.drupal.org/project/entity_browser/issues/3036556 📌 Fix the issues reported by phpcs RTBC ) rest all the pipeline is clean.

Note: I have added SKIP_ESLINT to avoid any functional failures due to Javascript (I think it is ideal to cover this separately)
Also 1 suggestion should we bump drupal version to 10.3 as we fixed deprecation here: https://git.drupalcode.org/project/entity_browser/-/merge_requests/97/di...

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

ankitv18 → created an issue.

ankitv18 → created an issue.

Anything pending here? or can we mark this as Fixed?

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

MR is ready for a review.

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

MR!9 is ready for a review.

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

Thanks @traviscarden,
I'll validate again with 1.0.x-dev branch to check whether this issue still there.

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

:( Yep isn't reflecting on my profile.

Hi @gagosha
Seems you missed to add credit 😅

MR!28 is ready for a review.

MR!30 is ready for a review

Plugin Id is general whereas as title is General MCP and this plugin generally returns the site information.
New plugin name suggestion

1. SiteInformation
2. SiteInfoMcp
3. GeneralSiteInfo
4. BasicSiteInfoMcp

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

MR!24 is ready for a review.

Merge this one first then take in consideration to merge https://www.drupal.org/project/metatag_ai/issues/3545322 📌 Add autowire to services Active

MR!23 is ready for a review.

ankitv18 → created an issue.

ankitv18 → created an issue.

ankitv18 → created an issue.

ankitv18 → created an issue.

PHPCS CI pipeline is clean now, MR!20 is ready for a review.

ankitv18 → created an issue.

Hi @nx2611
It seems you forgot to add credit on this issue 😄

MR!19 is ready for a review, Cspell is passing now

ankitv18 → created an issue.

MR!18 is ready for a review!!

ankitv18 → changed the visibility of the branch 3543530-missing-api-key to hidden.

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

ankitv18 → created an issue.

Hi @traviscarden,
I've validated the functionality of this module and found few issues on sorting tab.
When I add the moderation_state
Drupal\Core\Entity\Query\QueryException: 'moderation_state' not found in Drupal\Core\Entity\Query\Sql\Tables->ensureEntityTable() (line 373 of /var/www/html/docroot/core/lib/Drupal/Core/Entity/Query/Sql/Tables.php).

When I add the UUID or Path field getting 400 error in response.
Drupal\Core\Http\Exception\CacheableBadRequestHttpException: Invalid nested filtering. The field `uuid`, given in the path `uuid`, does not exist. in Drupal\jsonapi\Context\FieldResolver->resolveInternalEntityQueryPath() (line 292 of /var/www/html/docroot/core/modules/jsonapi/src/Context/FieldResolver.php).

I have included the MR!5 of https://www.drupal.org/project/jsonapi_query_builder/issues/3529219 ✨ Replace JSON:API Schema dependency with Open API JSON:ARG module Active and then tried to reproduce the issue but it working fine now

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

I created a symlink to version 5 of gin at my local to validate it using MR!5 of issue https://www.drupal.org/project/jsonapi_query_builder/issues/3529219 ✨ Replace JSON:API Schema dependency with Open API JSON:ARG module Active .

Adding screenshot for your reference

Changes looks good and all pipelines are green now.

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

Then in that case we should not enable previous major and if we want to run on previous major then we should restrict the module to install on php 8.3+ application.

MR!176 is ready for a review

Covered in https://www.drupal.org/project/ai_agents/issues/3542950 📌 Add PHP 8.1 tests Active

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

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

Hi @danrod,
Maintainer section doesn't have a proper format. See

MR!15 is ready for a review.

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

MR!14 is a ready for a review.

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

ankitv18 → changed the visibility of the branch 3542913-integration-with-key to hidden.

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

Looking good now ~~ Moving to RTBC

Good to be merge now ~~ moving into RTBC

@danrod,
Fixed the pipeline but Readme still needs some work like for Maintainers section we need to do the proper formatting [Author](URL)

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

Looks good and it would be great if we fix this also

[The Jonthornton jQuery Timepicker](https://github.com/jonthornton/jquery
-timepicker)
[jQuery Timepicker Addon](https://github.com/trentrichardson/jQuery
-Timepicker-Addon)

Check the screenshot how it is looking in the https://readme.so/editor

ankitv18 → changed the visibility of the branch 3536086-fix-validate-pipeline to hidden.

Covered in https://www.drupal.org/project/acquia_optimize/issues/3538217 ✨ Node edit page Acquia Optimize Functional restructure Active

Covered in https://www.drupal.org/project/acquia_optimize/issues/3538217 ✨ Node edit page Acquia Optimize Functional restructure Active

Changes looks good ~~ Pipeline are passing , CSpell is still having some smell which I can guess cover in this only.
Keeping this in NR for other to take a look.