Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Breadcrumb not showing for news article #740

Open
wants to merge 11 commits into
base: 7.3.x
Choose a base branch
from

Conversation

nissen-chris
Copy link
Contributor

@nissen-chris nissen-chris commented Aug 1, 2023

Issue link on drupal.org: 3378438

This issue is not ready for review!

Make sure these boxes are checked before submitting your pull request - thank you!

  • All coding styles are fulfilled. (How to check for cs issues?)
  • All tests are running locally. (How to run the test?)
  • Necessary update hooks are provided.
  • User roles have correct access for new introduced permission.
  • Every thunder module has a README.md in its root. Follow this guidelines, but we don't need every topic.
  • Code is covered with well-balanced amount of inline comments.
  • New features or changes are documented.

If you are really awesome, then your feature is covered by additional tests. Well done!

@nissen-chris nissen-chris self-assigned this Aug 1, 2023
@ol0lll ol0lll temporarily deployed to thunder-pr-740 August 1, 2023 14:02 Inactive
@ol0lll ol0lll temporarily deployed to thunder-pr-740 August 2, 2023 07:12 Inactive
@chrfritsch chrfritsch force-pushed the fix/breadcrumbs-news-article branch from 4a2086b to b2bf3fc Compare April 25, 2024 10:07
@chrfritsch chrfritsch changed the base branch from 6.5.x to 7.2.x April 25, 2024 10:11
@chrfritsch chrfritsch marked this pull request as ready for review April 25, 2024 14:27
@chrfritsch chrfritsch requested a review from IT-Cru June 17, 2024 13:43
@IT-Cru IT-Cru self-assigned this Jun 24, 2024
@dbosen dbosen changed the base branch from 7.2.x to 7.3.x September 5, 2024 07:31
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.

5 participants