🇮🇳India @Ishani Patel

Account created on 13 June 2022, over 3 years ago
  • Drupal Backend Developer at QED42 
#

Recent comments

🇮🇳India Ishani Patel

Hello, @joachim
I've created MR.
Kindly check.

🇮🇳India Ishani Patel

Hello @trickfun,
I've raised MR.
Please check and review.

Thank you!

🇮🇳India Ishani Patel

ishani patel made their first commit to this issue’s fork.

🇮🇳India Ishani Patel

I've updated MR.
PLease check.

Thank you!

🇮🇳India Ishani Patel

Hello @avpaderno,
I've raised MR,
Kindly check and review.

Thank You!

🇮🇳India Ishani Patel

ishani patel made their first commit to this issue’s fork.

🇮🇳India Ishani Patel

Hello @avpaderno,
I've updated the documentation comment and raised MR.
Please review.

Thank you!

🇮🇳India Ishani Patel

ishani patel made their first commit to this issue’s fork.

🇮🇳India Ishani Patel

Hello @avpaderno,
I got your point and updated MR.
Kindly check and let me know if any changes are required.

Thank you!

🇮🇳India Ishani Patel

I've removed an outdated @todo and raised MR
Thank you!

🇮🇳India Ishani Patel

ishani patel made their first commit to this issue’s fork.

🇮🇳India Ishani Patel

Hello @avpaderno,
I've raised MR,
Please review.

Thank you!

🇮🇳India Ishani Patel

ishani patel made their first commit to this issue’s fork.

🇮🇳India Ishani Patel

ishani patel made their first commit to this issue’s fork.

🇮🇳India Ishani Patel

I've checked with Drupal 11.2.3 and not replicated this issue.

🇮🇳India Ishani Patel

Yes, I've been facing the same issue, and when I update with this, it's working properly
public function init(ViewExecutable $view, DisplayPluginBase $display, ?array $options = NULL) {

🇮🇳India Ishani Patel

Hello @shamir.vs,
I've raised MR, Kindly check and review.

Thank you

🇮🇳India Ishani Patel

ishani patel made their first commit to this issue’s fork.

🇮🇳India Ishani Patel

Hello @vmarchuk,
I've raised MR,
Kindly check and let me know if any changes are required.

Thank you!

🇮🇳India Ishani Patel

ishani patel made their first commit to this issue’s fork.

🇮🇳India Ishani Patel

Hello @avpaderno,
I've updated MR as per your suggestion.
Please check and let me know if any changes are required.

Thank you!

🇮🇳India Ishani Patel

ishani patel made their first commit to this issue’s fork.

🇮🇳India Ishani Patel

Hello @joachim,
I've created MR,
Kindly check and review.

Thank you!

🇮🇳India Ishani Patel

Hello @avpaderno,
I've raised MR,
Kindly check and review.

Thank you!

🇮🇳India Ishani Patel

ishani patel made their first commit to this issue’s fork.

🇮🇳India Ishani Patel

Hello @aalin,
I've raised MR.
Kindly review it.

Thank you!

🇮🇳India Ishani Patel

Hello @damienmckenna,
I've raised MR.
Kindly check.

Thank you!

🇮🇳India Ishani Patel

I've raised MR,
Kindly check and review.

Thank you!

🇮🇳India Ishani Patel

Hello @bbrala,

Please check the code below and verify as I update those sections to use string_parts instead of string + regex:

core.entity_view_mode.*.*:
  type: config_entity
  label: 'Entity view mode settings'
  mapping:
    id:
      type: string_parts
      label: 'ID'
      # View mode IDs are in the form: entity_type.view_mode
      part_types:
        - type: string
          label: 'Entity type'
          pattern: '^[a-z0-9_]+$'
        - type: string
          label: 'View mode'
          pattern: '^[a-z0-9_]+$'

core.entity_form_mode.*.*:
  type: config_entity
  label: 'Entity form mode settings'
  constraints:
    FullyValidatable: ~
  mapping:
    id:
      type: string_parts
      label: 'ID'
      # Form mode IDs are in the form: entity_type.form_mode
      part_types:
        - type: string
          label: 'Entity type'
          pattern: '^[a-z0-9_]+$'
        - type: string
          label: 'Form mode'
          pattern: '^[a-z0-9_]+$'

core.entity_view_display.*.*.*:
  type: config_entity
  label: 'Entity display'
  mapping:
    id:
      type: string_parts
      label: 'ID'
      # Display IDs are in the form: entity_type.bundle.view_mode
      part_types:
        - type: string
          label: 'Entity type'
          pattern: '^[a-z0-9_]+$'
        - type: string
          label: 'Bundle'
          pattern: '^[a-z0-9_]+$'
        - type: string
          label: 'View mode'
          pattern: '^[a-z0-9_]+$'

core.entity_form_display.*.*.*:
  type: config_entity
  label: 'Entity form display'
  mapping:
    id:
      type: string_parts
      label: 'ID'
      # Form display IDs are in the form: entity_type.bundle.form_mode
      part_types:
        - type: string
          label: 'Entity type'
          pattern: '^[a-z0-9_]+$'
        - type: string
          label: 'Bundle'
          pattern: '^[a-z0-9_]+$'
        - type: string
          label: 'Form mode'
          pattern: '^[a-z0-9_]+$'

Let me know your thoughts.

Thanks!

🇮🇳India Ishani Patel

I've raised MR from the patch.
Please review.
THank you!

🇮🇳India Ishani Patel

Hello @astonvictor,
I've raised MR. Please review.
Thanks.

🇮🇳India Ishani Patel

Hello, @anaconda777
You need to import the correct class from the OpenAI provider module.
At the top of your file (modules/contrib/ai_provider_vllm/src/Plugin/AiProvider/VllmProvider.php),
Add:
use Drupal\ai_provider_openai\OpenAiChatMessageIterator;

Thank you!

🇮🇳India Ishani Patel

Hello, @marcus_johansson
I've added new properties and getters/setters, called that method, and also updated StreamedChatMessageIteratorInterface.

Please check and review the MR.
Let me know if any changes are required.

Thank you!

🇮🇳India Ishani Patel

Hello,
I've created a Merge request.
Please check and review.

Thank you!

🇮🇳India Ishani Patel

I'm facing the same issue when I select a provider in the Embeddings Generator Explorer.

🇮🇳India Ishani Patel

I've created PR.
Kindly check and review it.

🇮🇳India Ishani Patel

Hello @joachim,
I've updated the text, please check and review.

Thank you!

🇮🇳India Ishani Patel

Hello @prudloff,
Thanks for the review, and I updated the MR as per your suggestion.
Kindly check.

Thank you!

🇮🇳India Ishani Patel

I've tested with D10.4.5 and D11.1.6 ,checked with adding new language also checked with already added language edit page but not reproduced this error.

Thank you!

🇮🇳India Ishani Patel

Hello,
I've followed the steps and checked with D11.1.6, and using PHP Version 8.3.19
But not reproduce the issue.

Thank you!

🇮🇳India Ishani Patel

Hello,
I've created MR,
Kindly review.

Thank you!

🇮🇳India Ishani Patel

Hello,
I've followed the steps as mentioned and checked with D10.4 and D11. x But I can't replicate this issue.

Thank you!

🇮🇳India Ishani Patel

Hello,
I've also tried with D10.4 and 1.1.x of AI and sub modules, but no luck.

🇮🇳India Ishani Patel

Hello @rosemaryreilman,
I've some solution regarding this error.
Please check the details below.

1. Check Field Configuration in Active Config
- Using this command: drush config:grep field_vt_recipients
=> If they still exist, remove them:
- drush config:delete field.field.monument.field_vt_recipients
- drush config:delete field.storage.monument.field_vt_recipients

2. Also, you can check for Field Definitions in the Database:
- Using query like this: SELECT * FROM config WHERE name LIKE '%field_vt_recipients%';

Hope this solution is helpful for you.
Thank you!

🇮🇳India Ishani Patel

Hello,
I've created an MR from the patch (Since patches are no longer tested, a merge request needs to be provided)

Thank you!

🇮🇳India Ishani Patel

ishani patel made their first commit to this issue’s fork.

🇮🇳India Ishani Patel

Hello,
I've again tested this issue with 1.1.x and it's working like when I created fresh node and go to the translate tab, That page show me error
"Please set a default provider & model for the text translation the AI module settings." If I not set the default provider and model
So it's working correct.
Please checked the attach screencast for the same.

Working Video Link: https://www.loom.com/share/c7d5f508c9114c3a812c34a3a7a18928

Thank you!

🇮🇳India Ishani Patel

Hello,
I've followed the mentioned steps and replicated the issue.
After applying the MR changes and no Ajax issues were faced in the console.
Please follow the screenshot below for reference.

Before Screenshot:

After Screenshot:

Thank you!

🇮🇳India Ishani Patel

This is a duplicate issue of https://www.drupal.org/project/ai/issues/3521769 🐛 AI Settings - Default Model needs refresh when Default Provider changes Active
So I closed this.

🇮🇳India Ishani Patel

Hello @gkffzs,
I've fixed PHPCS and phpstan errors.

Kindly review it.

Thank you!

🇮🇳India Ishani Patel

Hello,
I've created MR.
Kindly review.

Thank You!

Production build 0.71.5 2024