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

YSP-683: ONHA program and news migration #837

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Conversation

nJim
Copy link
Contributor

@nJim nJim commented Jan 6, 2025

YSP-683: ONHA program and news migration

Description of work

  • Includes verbose comments and readme to document this migration and to serve as a reference for future migrations.
  • Defines migrations for news and program related content for the Office of New Haven Affairs. All migrations are prefixed with onha as they include opinionated decisions that may not apply to future migrations.

Functional testing steps:

  • This work is already complete. This PR simply adds the documentation and reference files to the upstream repo.
  • These migrations were executed on the ONHA Development multisite.

@nJim nJim requested a review from a team as a code owner January 6, 2025 14:12
@nJim nJim changed the title feat(YSP-683): define ONHA migration YSP-683: ONHA program and news migration Jan 6, 2025
@dblanken-yale
Copy link
Contributor

@nJim Would it mess up Drupal at all to compartmentalize these into a subfolder? Was thinking a folder for ONHA, a folder for the next one, so it'll be easier to see what was done without seeing a ton of folders. But is that as easy as just creating the folder and Drupal will still pick them all up or will it cause any issues?

@dblanken-yale
Copy link
Contributor

@nJim Any more work to be done in here or is it ready for a merge? I thought there was a bug fix but didn't see it in the commits, so wanted to make sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants