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

Closes #3086 Adding demo content for publications #3585

Merged
merged 13 commits into from
Oct 18, 2024
Merged

Closes #3086 Adding demo content for publications #3585

merged 13 commits into from
Oct 18, 2024

Conversation

trackleft
Copy link
Member

@trackleft trackleft commented Jul 31, 2024

Description

This PR adds support for managing publications in the az_demo module. It includes the following changes:

  • Added az_publication as a dependency in az_demo.info.yml.
  • Created a new block configuration for displaying publications authored by a person.
  • Added JSON data for demo authors and publications.
  • Added a new menu link for the publications page.
  • Added a new node for the publications page.
  • Updated existing person nodes to include netid.
  • Added new taxonomy terms for publication categories.
  • Created migration configurations for demo authors, publications, and publication categories.

Related issues

Closes #3086

How to test

  1. Enable the az_demo module and ensure all dependencies are satisfied.
  2. Verify that the publications page is accessible at /publications and displays the correct content.
  3. Check the person nodes to ensure the netid field is populated correctly.
  4. Verify the new block configuration for publications is functioning as expected.

Types of changes

Arizona Quickstart (install profile, custom modules, custom theme)

  • Patch release changes
    • Bug fix
    • Accessibility, performance, or security improvement
    • Critical institutional link or brand change
    • Adding experimental module
    • Update experimental module
  • Minor release changes
    • New feature
    • Breaking or visual change to existing behavior
    • Upgrade experimental module to stable
    • Enable existing module by default or database update
    • Non-critical brand change
    • New internal API or API improvement with backwards compatibility
    • Risky or disruptive cleanup to comply with coding standards
    • High-risk or disruptive change (requires upgrade path, risks regression, etc.)
  • Other or unknown
    • Other or unknown

Drupal core

  • Patch release changes
    • Security update
    • Patch level release (non-security bug-fix release)
    • Patch removal that's no longer necessary
  • Minor release changes
    • Major or minor level update
  • Other or unknown
    • Other or unknown

Drupal contrib projects

  • Patch release changes
    • Security update
    • Patch or minor level update
    • Add new module
    • Patch removal that's no longer necessary
  • Minor release changes
    • Major level update
  • Other or unknown
    • Other or unknown

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • My change requires release notes.

@trackleft trackleft requested a review from a team as a code owner July 31, 2024 22:46
@trackleft trackleft added enhancement New feature or request publications labels Jul 31, 2024
@joeparsons joeparsons changed the title Adding demo content for publications Closes #3086 Adding demo content for publications Jul 31, 2024
@joeparsons
Copy link
Member

Question to consider: should this be a 2.12.0 change?

@trackleft
Copy link
Member Author

Question to consider: should this be a 2.12.0 change?

Sure, I just want more demo content so I can demo upcoming work more easily.

mmunro-ltrr
mmunro-ltrr previously approved these changes Aug 1, 2024
Copy link
Member

@mmunro-ltrr mmunro-ltrr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on a local build. Looking good!

mmunro-ltrr
mmunro-ltrr previously approved these changes Aug 2, 2024
joeparsons
joeparsons previously approved these changes Aug 9, 2024
@trackleft trackleft marked this pull request as draft October 17, 2024 23:37
@trackleft trackleft marked this pull request as ready for review October 18, 2024 00:12
joeparsons
joeparsons previously approved these changes Oct 18, 2024
mmunro-ltrr
mmunro-ltrr previously approved these changes Oct 18, 2024
@trackleft trackleft dismissed stale reviews from mmunro-ltrr and joeparsons via 9a6478a October 18, 2024 18:02
@joeparsons joeparsons merged commit 22a9860 into main Oct 18, 2024
16 checks passed
@joeparsons joeparsons deleted the issue/3086 branch October 18, 2024 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Add publication demo content
5 participants