Skip to content

feat(seed): populate homepage config, carousel, cta action, and code of conduct - #65

Merged
ASPactores merged 1 commit into
mainfrom
feat/39-populate-homepage-config-carousel-cta-code-of-conduct
Sep 11, 2026
Merged

ASPactores merged 1 commit into
mainfrom
feat/39-populate-homepage-config-carousel-cta-code-of-conduct

Conversation

@alasdiel

@alasdiel alasdiel commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

📝 Overview

What does this PR do? Briefly describe the changes and the goal of this PR.

This PR populates the CMS using assets from the DurianPy website.

Aims to resolve:
https://github.com/DurianPy-Davao-Python-User-Group/Engineering-Board/issues/39

🚀 Type of Change

  • 🐛 Bug Fix: Fixed something that was broken.
  • ✨ New Feature: Added new functionality or pages.
  • 🗄️ CMS / Database: Updated collections, fields, or config in Payload CMS.
  • 🧹 Chore / Clean up: Text updates, typos, or code cleanup.

🧱 CMS & Database Changes

Skip this section if no CMS or database changes were made.

  • Migration Created: Ran payload migrate:create and saved the file.
  • Local Verification: Verified changes in the local admin panel.

📊 Data Structure

Describe the new fields/tables or drag-and-drop a screenshot here:


🧪 Testing & Validation

  • Test Case Approval: Verified that test cases were commented on the assigned GitHub issue and approved by the Project Lead before development started.
  • Unit Tests: Added or updated unit test scripts covering the new logic.
  • Admin Panel: Verified behavior in the CMS Admin UI.
  • API Check: Tested REST/GraphQL endpoints for correct data.

📸 Visual Evidence (Mandatory)

Important

Visual proof is required to demonstrate that the approved test cases from the issue were successfully met.
Please provide screenshots, screen recordings (GIFs/Videos), or sample API outputs proving that the feature satisfies all approved test cases and do not introduce regressions.

image

📋 Final Checklist

  • Sync: Rebased from main and resolved conflicts.
  • Pre-commit: Installed and ran pre-commit hooks successfully.
  • Security: Verified no secrets or local credentials are included.
  • Build: Project builds without errors.
  • Visuals: Attached mandatory visual evidence proving all test cases passed.
  • Migrations: Migration script included (if applicable).

🧠 Extra Notes / Questions

Add any additional context or questions for the reviewers here.

@alasdiel
alasdiel requested a review from a team September 2, 2026 03:20

@ASPactores ASPactores left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@ASPactores
ASPactores merged commit ee1c05e into main Sep 11, 2026
5 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.

2 participants