Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Changes for project seeds to work with knex seed file. #8877

Merged
merged 1 commit into from
Sep 22, 2023
Merged

Conversation

hanzlamateen
Copy link
Member

@hanzlamateen hanzlamateen commented Sep 22, 2023

Summary

Made these changes to add support for knex based seed files in projects. Also cleaned up old sequelized based seeds.

🤖 Generated by Copilot at 90a0387

Migrated media, networking, and project services from Sequelize to Knex as the database query builder and seeder. Removed unused Sequelize seeds and logic from seeder.ts and seeder-config.ts files. Updated ProjectConfigInterface and seed arrays to use the KnexSeed interface.

References

#7645

Explanation

🤖 Generated by Copilot at 90a0387

  • Migrated the database seeding from Sequelize to Knex (link, link, link, link, link, link, link, link, link)
  • Updated the databaseSeed property of the ProjectConfigInterface in ProjectConfigInterface.ts to use the KnexSeed interface (link)
  • Updated the mediaSeeds and networkingSeeds arrays in seeder-config.ts to use the KnexSeed interface (link, link)
  • Removed the import and usage of the ServicesSeedConfig interface from seeder-config.ts and seeder.ts (link, link)
  • Removed the sequelizeSeeds array from seeder-config.ts and the loop that seeded the database using Sequelize from seeder.ts (link, link)
  • Included the mediaSeeds, networkingSeeds, and installedProjects arrays in the knexSeeds array in seeder-config.ts (link)
  • Deleted the ServicesSeedConfig.ts file from the common package (link)

🤖 Generated by Copilot at 90a0387

Sequelize is gone
Knex seeds the database now
Winter of changes

QA Steps

List any additional steps required to QA the changes of this PR, as well as any supplemental images or videos.

Checklist

  • If this PR is still a WIP, convert to a draft
  • When this PR is ready, mark it as "Ready for review"
  • ensure all checks pass
  • Changes have been manually QA'd
  • Changes reviewed by at least 2 approved reviewers

@hanzlamateen hanzlamateen marked this pull request as draft September 22, 2023 12:09
@hanzlamateen hanzlamateen marked this pull request as ready for review September 22, 2023 16:52
@barankyle barankyle added this pull request to the merge queue Sep 22, 2023
Merged via the queue into dev with commit 5bcdde4 Sep 22, 2023
@barankyle barankyle deleted the project-seeds branch September 22, 2023 17:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants