@/CivicSignalBlog - Add Ability to Manage MediaData Page #931

m453h · 2024-10-03T11:36:34Z

Description

This PR adds the ability to Manage MediaData Page, the content managed here will be consumed by the MediaData Page proposed in this PR

Fixes:

Type of change

New feature (non-breaking change which adds functionality)
This change requires a documentation update

Screenshots

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

…ections

…o read content from the CMS

kilemensi

👍🏽

apps/civicsignalblog/src/payload/access/canAccessApplication.js

apps/civicsignalblog/src/payload/blocks/WebTools/PageHeader.js

apps/civicsignalblog/src/payload/blocks/WebTools/ResourceList.js

Co-authored-by: Clemence Kyara <[email protected]>

…eForAfrica/ui into feat/civicsignal_webtools_media_data

…hBlog PageHeader

…eForAfrica/ui into feat/civicsignal_webtools_media_data

kilemensi

👍🏽

apps/civicsignalblog/src/components/PageHeader/PageHeader.js

apps/civicsignalblog/src/payload/access/canAccessApplication.js

apps/civicsignalblog/src/payload/lib/data/common/applications.js

…admin users

kilemensi

Almost there... 👍🏽

apps/civicsignalblog/src/payload/collections/Users.js

apps/civicsignalblog/payload.config.ts

apps/civicsignalblog/src/components/PageHeader/PageHeader.js

apps/civicsignalblog/src/payload/access/canAccessApplication.js

apps/civicsignalblog/src/payload/collections/Main/MediaData.js

Co-authored-by: Clemence Kyara <[email protected]>

… PageHeader

kilemensi

Code LGTM!

apps/civicsignalblog/.eslintrc.js

apps/civicsignalblog/payload.config.ts

apps/civicsignalblog/src/components/PageHeader/PageHeader.js

apps/civicsignalblog/src/payload/access/canAccessApplication.js

apps/civicsignalblog/src/payload/blocks/PageHeader.js

apps/civicsignalblog/src/payload/collections/Main/MediaData.js

apps/civicsignalblog/src/payload/components/allowedAppSelect/index.tsx

…ndex.tsx Co-authored-by: Clemence Kyara <[email protected]>

…_webtools_media_data

Co-authored-by: Clemence Kyara <[email protected]>

m453h added 4 commits October 2, 2024 08:18

chore: Include SEO fields in application specific pages and post coll…

1555c16

…ections

feat: Implement MediaData Collection

a13a620

feat: Implement PageHeader block for Research and WebTools Pages

dcc287f

chore: Rename common applications and allow non authenticated users t…

90ace97

…o read content from the CMS

m453h added the enhancement New feature or request label Oct 3, 2024

m453h self-assigned this Oct 3, 2024

m453h and others added 3 commits October 4, 2024 12:40

chore: Add Document upload feature for Media Data

c59dc96

feat: Add ResourceList Block to civicsignal tools pages

d968728

Merge branch 'main' into feat/civicsignal_webtools_media_data

9dda643

m453h marked this pull request as ready for review October 8, 2024 05:51

m453h requested a review from a team October 8, 2024 05:51

kilemensi reviewed Oct 8, 2024

View reviewed changes

m453h and others added 5 commits October 8, 2024 14:31

Update apps/civicsignalblog/src/payload/blocks/WebTools/ResourceList.js

9b58733

Co-authored-by: Clemence Kyara <[email protected]>

Merge branch 'main' into feat/civicsignal_webtools_media_data

213f029

Merge branch 'feat/civicsignal_webtools_media_data' of github.com:Cod…

d7da82b

…eForAfrica/ui into feat/civicsignal_webtools_media_data

feat: Add API key authentication in users collection

87182e3

refractor: Use shared page header and add RichText support to Researc…

3c0468c

…hBlog PageHeader

m453h requested a review from kilemensi October 15, 2024 10:41

m453h and others added 3 commits October 15, 2024 13:42

Merge branch 'main' into feat/civicsignal_webtools_media_data

0284159

chore: Update Jest snapshots after adding RichText support in PageHeader

5ec8a10

Merge branch 'feat/civicsignal_webtools_media_data' of github.com:Cod…

fcb4b65

…eForAfrica/ui into feat/civicsignal_webtools_media_data

kilemensi reviewed Oct 15, 2024

View reviewed changes

apps/civicsignalblog/src/components/PageHeader/PageHeader.js Outdated Show resolved Hide resolved

apps/civicsignalblog/src/payload/access/canAccessApplication.js Outdated Show resolved Hide resolved

apps/civicsignalblog/src/payload/lib/data/common/applications.js Show resolved Hide resolved

m453h added 8 commits October 16, 2024 15:56

chore: Merge main into feat/civicsignal_webtools_media_data

eb85cb4

chore: Use RichText component from @commons-ui/payload

2b1c09d

chore: Update Jest snapshots for CMSContent and LongFormRichText

c785453

chore: Create Subscriber role

6379219

feat: Add check on user allowed apps

7452b85

feat: Update app selection component to only include allowed apps

4694601

feat: Restrict changing of default, current and allowed apps for non …

9167015

…admin users

chore: Implement access control for collections

5809930

m453h requested a review from kilemensi October 18, 2024 07:55

fix: Add custom x-current-app header to list of allowed headers

0e6586c

kilemensi reviewed Oct 18, 2024

View reviewed changes

chore: Fix formatting issues

b2a186c

m453h force-pushed the feat/civicsignal_webtools_media_data branch from cda151c to b2a186c Compare October 18, 2024 11:11

m453h and others added 6 commits October 18, 2024 14:11

Update apps/civicsignalblog/payload.config.ts

8642b72

Co-authored-by: Clemence Kyara <[email protected]>

Update apps/civicsignalblog/src/payload/access/canAccessApplication.js

9be202d

Co-authored-by: Clemence Kyara <[email protected]>

Update apps/civicsignalblog/src/payload/collections/Main/MediaData.js

0042654

Co-authored-by: Clemence Kyara <[email protected]>

chore: Improve field names in MediaData and removed unecessary ref in…

05a8c47

… PageHeader

feat: Implement Custom Allowed App Selection field

8fb9931

fix: Fix failed build due to missing TypeScript parser

279b57d

m453h requested a review from kilemensi October 22, 2024 11:28

kilemensi approved these changes Oct 22, 2024

View reviewed changes

Update apps/civicsignalblog/src/payload/components/allowedAppSelect/i…

0d92d37

…ndex.tsx Co-authored-by: Clemence Kyara <[email protected]>

koechkevin approved these changes Oct 23, 2024

View reviewed changes

m453h and others added 9 commits October 23, 2024 12:53

chore: Add comment describing the need for typescript-eslint/parser

31bfee8

chore: Fix merge conflict after merging changes from feat/civicsignal…

999107d

…_webtools_media_data

Update apps/civicsignalblog/payload.config.ts

412826e

Co-authored-by: Clemence Kyara <[email protected]>

chore: Improve handling of custom headers and clean up eslint warnings

3b7a0ea

feat: Explicitly set leaves supported by SlateEditor

5cb5e49

Merge branch 'main' into feat/civicsignal_webtools_media_data

3a37f14

chore: Fix merge conflicts on pnpm-lock.yaml

d4d2aea

chore: Remove typescript parser

1ed6712

Merge branch 'main' into feat/civicsignal_webtools_media_data

ee84370

m453h added this pull request to the merge queue Oct 25, 2024

Merged via the queue into main with commit 74aeef4 Oct 25, 2024
2 of 3 checks passed

m453h deleted the feat/civicsignal_webtools_media_data branch October 25, 2024 07:34

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

@/CivicSignalBlog - Add Ability to Manage MediaData Page #931

@/CivicSignalBlog - Add Ability to Manage MediaData Page #931

m453h commented Oct 3, 2024

kilemensi left a comment

kilemensi left a comment

kilemensi left a comment

kilemensi left a comment

@/CivicSignalBlog - Add Ability to Manage MediaData Page #931

@/CivicSignalBlog - Add Ability to Manage MediaData Page #931

Conversation

m453h commented Oct 3, 2024

Description

Type of change

Screenshots

Checklist:

kilemensi left a comment

Choose a reason for hiding this comment

kilemensi left a comment

Choose a reason for hiding this comment

kilemensi left a comment

Choose a reason for hiding this comment

kilemensi left a comment

Choose a reason for hiding this comment