Skip to content

Eleino/refactor/624 refactor prg page data sourcing via directus - #691

Open
eleino wants to merge 4 commits into
devfrom
eleino/refactor/624-refactor-prg-page-data-sourcing-via-directus
Open

eleino wants to merge 4 commits into
devfrom
eleino/refactor/624-refactor-prg-page-data-sourcing-via-directus

Conversation

@eleino

@eleino eleino commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

📄 Pull Request Overview

Closes 624

🔧 Changes Made

  1. Created prg_page, prg_documents, and prg_board_members collections in Directus.
  2. Added functions to frontend for fetching the data from Directus, and mapping the data from the three collections into a single object.
  3. Updated PrgPage to use the data fetched from Directus instead of hardcoded data. The UI was not changed except to add loading/error information.
  4. Removed hardcoded data from frontend including now unused images.

✅ Checklist Before Submission

  • Functionality: I have tested my code, and it works as expected.
  • JSDoc: I have added or updated JSDoc comments for all relevant code.
  • Debugging: No console.log() or other debugging statements are left.
  • Clean Code: Removed commented-out or unnecessary code.
  • Tests: Added new tests or updated existing ones for the changes made.
  • Documentation: Documentation has been updated (if applicable).

📝 Additional Information

Provide any additional context or information that reviewers may need to know:

  • Screenshots: [Include any screenshots or videos if the changes affect the UI]
  • Dependencies: [Mention any new dependencies or breaking changes]
  • Known Issues: [List any known issues or limitations]

@codecov-alt

codecov-alt Bot commented Sep 30, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 24.17582% with 69 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...-next-migration/src/entities/Prg/api/prgPageApi.ts 0.00% 33 Missing ⚠️
...end-next-migration/src/entities/Prg/api/mappers.ts 0.00% 22 Missing ⚠️
...igration/src/entities/Prg/api/useGetPrgPageData.ts 0.00% 10 Missing ⚠️
...migration/src/preparedPages/PRGPage/ui/PRGPage.tsx 88.00% 3 Missing ⚠️
frontend-next-migration/src/entities/Prg/index.ts 0.00% 1 Missing ⚠️
Files with missing lines Coverage Δ
...-migration/src/shared/appLinks/appExternalLinks.ts 100.00% <ø> (ø)
frontend-next-migration/src/entities/Prg/index.ts 0.00% <0.00%> (ø)
...migration/src/preparedPages/PRGPage/ui/PRGPage.tsx 92.72% <88.00%> (-4.90%) ⬇️
...igration/src/entities/Prg/api/useGetPrgPageData.ts 0.00% <0.00%> (ø)
...end-next-migration/src/entities/Prg/api/mappers.ts 0.00% <0.00%> (ø)
...-next-migration/src/entities/Prg/api/prgPageApi.ts 0.00% <0.00%> (ø)

... and 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@eleino eleino linked an issue Sep 30, 2026 that may be closed by this pull request
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.

Refactor PRG page data sourcing via Directus

1 participant