Skip to content

Conversation

@mglaman
Copy link
Contributor

@mglaman mglaman commented Dec 7, 2023

Fixes #599 #282, dependencies in next_entity_type_config for the entity type and bundle being targeted.

This pull request is for: (mark with an "x")

  • examples/*
  • modules/next
  • packages/next-drupal
  • starters/basic-starter
  • starters/graphql-starter
  • Other

GitHub Issue: #599

  • I need help adding tests. (mark with an "x")
    Code changes need test coverage. If you don't know
    how to make tests, check this box to ask for help.

Describe your changes

Adds configuration dependency to target entity type and bundle

@vercel
Copy link

vercel bot commented Dec 7, 2023

@mglaman is attempting to deploy a commit to the Chapter Three Team on Vercel.

A member of the Team first needs to authorize it.

@mglaman
Copy link
Contributor Author

mglaman commented Feb 27, 2024

@JohnAlbin I tried running locally, but for some reason all tests fail now. So I wanted to use the CI... but it looks like next.yml workflow may need approval.

EDIT: needed to update drupal/next in the drupal/composer.lock

@mglaman
Copy link
Contributor Author

mglaman commented Feb 27, 2024

Okay, it should be all set, now

@bojanbogdanovic bojanbogdanovic force-pushed the NextEntityTypeConfig-config-dependencies branch 2 times, most recently from 99722b5 to 747b56d Compare November 3, 2025 08:18
@bojanbogdanovic bojanbogdanovic self-requested a review November 3, 2025 08:19
Fixes dependencies in next_entity_type_config for the entity type and bundle being targeted.
save test entity so dependencies are calculated
@bojanbogdanovic bojanbogdanovic force-pushed the NextEntityTypeConfig-config-dependencies branch from 747b56d to 317224a Compare December 18, 2025 15:49
@bojanbogdanovic bojanbogdanovic merged commit ed7fdd2 into chapter-three:main Dec 25, 2025
13 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NextEntityTypeConfig does not calculate dependencies

3 participants