Skip to content

Trustlab: Post Page add block support #1208

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

koechkevin
Copy link
Contributor

@koechkevin koechkevin commented Jun 25, 2025

Description

  • Implement PostImageOverview component with props for title, content, image, deadline, location, and application link.
  • Create corresponding Jest tests and snapshots for the new component.
  • Update pagify function to include new post type and format deadline in getPost utility.
  • Add PostImageContent block to the payload structure.
  • Enhance BaseContentCollection to support new content blocks.
  • Update index files to export new components and utilities.

Fixes #1177

Type of change

  • New feature (non-breaking change which adds functionality)

Screenshots

image

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation

- Implement PostImageOverview component with props for title, content, image, deadline, location, and application link.
- Create corresponding Jest tests and snapshots for the new component.
- Update pagify function to include new post type and format deadline in getPost utility.
- Add PostImageContent block to the payload structure.
- Enhance BaseContentCollection to support new content blocks.
- Update index files to export new components and utilities.
@koechkevin koechkevin self-assigned this Jun 25, 2025
@koechkevin koechkevin requested a review from a team June 25, 2025 09:47
@kilemensi
Copy link
Member

In between tasks @koechkevin but qq, is this (completely) different from the PageOverview component we had before?

@koechkevin
Copy link
Contributor Author

In between tasks @koechkevin but qq, is this (completely) different from the PageOverview component we had before?

Yes, completely different

@kilemensi
Copy link
Member

kilemensi commented Jun 25, 2025

🤔 ... it could be but lets double/triple check that and make sure it's not just design "bugs"/inconsistents

Desktop

D

Mobile

M

…text colors; update PageOverview payload to include color settings field
@kilemensi
Copy link
Member

Your reviews @CodeForAfrica/tech ?

PS: Could be unrelated @koechkevin but I see we are using applicationActive in this PR but isClosed is used in #1207; aren't these two props tracking the same thing?

…tionActive; modify getPost utility to reflect deadline status
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.

Incubator (Site Page)
2 participants