🇮🇳India @_pratik_

#27 seems working for 10.2.10 . Need to check if it pass test cases.

The commit in #42 would work for 10.2.10 also ?

Rerolled Patch for 10.2.10

i am using drupal 7 project and used composer to add fastglass/sendgrid library (composer require fastglass/sendgrid:~1.0).
Later pushed all files, and still getting error Class 'SendGrid\Client' not found in SendGridMailSystem->mail()

I got same issue where upgraded from D9 to D10.

Please try this patch.
Thanks

The above patch look good, it no phpcs error/warning found.
Thanks

Try using "path_alias" instead for former.
Thanks

site_factory.domain_negotiator:
class: 'Drupal\site_factory\DomainNegotiator'
arguments:
- '@request_stack'
- '@entity_type.manager'
- '@language_manager'
- '@path_alias'
- '@site_factory.site_manager'
tags:
- { name: path_processor_inbound, priority: 900 }
- { name: path_processor_outbound, priority: 400 }

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

Replaced in some files.
thanks

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

In response to #6.

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

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

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

Pushed with commit id 985ef95b

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

Some points from #19 are covered in patch.
Thanks

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

patch #30 was not applying, Attached rerolled patch.
Unable to find comments line from #20, #26
thanks

Attached interdiff between last two commits,
Thanks

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

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

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

#43 patch not applied for me also, Please try this new patch.
Thanks

#15 patch contains error, #16 is corrupted.
Try this.
Thanks

Addressed #75 comment.
Thanks

My bad, uploaded same patch as #43 in #44.

Fix for failures in #43
Thanks

Fix for mention in #23
thanks

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

CCF fix #23.

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

it seems drupal/component is not compatilble with d10.

The issue here is , route needs to be defined in routing.yml file.

redis.admin_display:
  path: 'redis/demo/page'
  defaults:
    _form: '\Drupal\redis\Form\RedisDemoForm'
    _title: 'Redis configuration'
  requirements:
    _access: 'TRUE'

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

Changes regarding #4, not sure on point 3.
Thanks