πŸ‡¨πŸ‡·Costa Rica @mcortes19

Account created on 3 November 2015, over 8 years ago
#

Merge Requests

Recent comments

πŸ‡¨πŸ‡·Costa Rica mcortes19

Merged to dev branch

πŸ‡¨πŸ‡·Costa Rica mcortes19

mcortes19 β†’ made their first commit to this issue’s fork.

πŸ‡¨πŸ‡·Costa Rica mcortes19

mcortes19 β†’ made their first commit to this issue’s fork.

πŸ‡¨πŸ‡·Costa Rica mcortes19

@almunnings yes, I will explore option 3 to expose it as MetaTagScript

πŸ‡¨πŸ‡·Costa Rica mcortes19

@almunnings Since we are using graphql-request for batching request, is not easy to replicate this issue outside the app.

After a thoroughly debugging, we noticed at some point the`EntityOwner` FieldType wasn't returning the `entity_owner` data-producer when we requested data using batch-request. So we ended up by removing that approach and just request all information we need in a single GraphQL query.

πŸ‡¨πŸ‡·Costa Rica mcortes19

@almunnings I confirm #8 works for me as well

πŸ‡¨πŸ‡·Costa Rica mcortes19

@almunnings We are migrating a site from D7 with a lot of Paragraphs and when we enabled some fields the error appeared, but was hard to identify which field or fields are the cause because was most a random behavior.

Definitely I'll debug thoroughly and posted here my finding. For now this MR fixes the issue!

πŸ‡¨πŸ‡·Costa Rica mcortes19

@abhishek_gupta1 I added a MR based on your solution

πŸ‡¨πŸ‡·Costa Rica mcortes19

I tested on the D10.8 site and it returned the following error: For error #0: Error: Call to protected method Drupal\menu_link_content\Plugin\Menu\MenuLinkContent::getEntity() from scope Drupal\graphql_compose_menus\Plugin\GraphQL\DataProducer\MenuTreeLinkEntity in Drupal\graphql_compose_menus\Plugin\GraphQL\DataProducer\MenuTreeLinkEntity->resolve() (line 45 of /app/web/modules/contrib/graphql_compose/modules/graphql_compose_menus/src/Plugin/GraphQL/DataProducer/MenuTreeLinkEntity.php)

Seems like the `getEntity()` method is public until D10.2 β†’ and many sites are not ready to move to D10.2

πŸ‡¨πŸ‡·Costa Rica mcortes19

Thanks @CedricL, I'll definitely use it as inspiration and hopefully as a basis for contributing.

πŸ‡¨πŸ‡·Costa Rica mcortes19

mcortes19 β†’ made their first commit to this issue’s fork.

πŸ‡¨πŸ‡·Costa Rica mcortes19

In group v3 the group_content was changed to β†’ group_relationship, also it requires the plugin_id field, so your migration could look like this:

source:
  plugin: d7_node
  node_type: oportunity
process:
  type:
    plugin: default_value
    default_value: program-group_node-opportunity
  plugin_id:
    plugin: default_value
    default_value: group_node:opportunity
  gid:
    plugin: migration_lookup
    migration: groups_audience
    no_stub: true
    source: groups_audience
  entity_id:
    plugin: migration_lookup
    migration: program_node_migration
    no_stub: true
    source: nid
destination:
  plugin: 'entity:group_relationship'
πŸ‡¨πŸ‡·Costa Rica mcortes19

Reviewing the code seems roles are first sorted by scope and then by their weight. not sure if this should change in the future to only use the weight sort

Production build 0.69.0 2024