Skip to content
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.

Sign up for GitHub

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

Jump to bottom

upgrade payload to ver. 3.0 #53

Merged
merged 57 commits into from
Oct 14, 2024
Merged

upgrade payload to ver. 3.0 #53

merged 57 commits into from
Oct 14, 2024

Conversation

rccsousa
Copy link
Collaborator

@rccsousa rccsousa commented Oct 10, 2024 •
edited
Loading

Why:

  • current version of the project is running on Payload 2.0 which required hosting Payload on a separate Express server;
  • the project wasn't building nor deploying to Netlify properly given this issue;

How:

  • upgraded the project dependencies and files to use Payload verion 3.0

Detailed update notes:

Changes:

  • Updated ESLint, TypeScript and NextJS config files to comply with Payload 3.0;
  • Updated existing collections to comply with Payload 3.0 configuration options.
  • Renamed podcast-episodes collection to podcasts.
  • Refactored content fetchers to use Payload Local API instead of GraphQL queries.
  • Refectored components that use the above mentioned fetchers to reflect changes;
  • Updated blogpost content to use Payload 3.0 Lexical Serializer, no longer relying on dangerouslySetInnerHTML.

Cleanup:

  • Removed deprecated Payload 2.0 files.
  • Moved Payload Admin Panel to the ./app folder, as it is now treated as a page by Next.js.

Roadmap:

  • Reorganize project files to improve navigation by separating Payload components from frontend components.

rccsousa added 27 commits October 9, 2024 14:59
@rccsousa
update project dependencies to satisfy payload 3.0 requirements
8f76197
@rccsousa
update next configs to satisfy payload 3.0
490e988
@rccsousa
freeze frontend to prepare cms admin panel rebuild
296e249
@rccsousa
reset payload and integrate with next app
86aaec5
@rccsousa
refactor front-end to comply with payload 3.0
574e386
@rccsousa
eject deprecated payload 2.0 files
f39e4ea
@rccsousa
fix: rename imports to comply with payload 3.0
16eafee
@rccsousa
purge unused files
98a6ed3
@rccsousa
freeze frontend
44dc2af 
- temporarily suspend imports while files are reorganize to attempt
  building
@rccsousa
purge unused files
540aa0f
@rccsousa
update next routes
440e726
@rccsousa
update in-component imports
481f3ca
@rccsousa
reset eslint
f93e292
@rccsousa
prepare refactor
c173708
@rccsousa
update content fetchers
8ba6f50 
- transition to payload local API
@rccsousa
refactor HubContentGrid
ddf13cf 
- renamed filterArticles to filterContent;
- renamed articles to content for consistency and clarity,
- updated typing,
@rccsousa
fix content grid navbar
142c99e
@rccsousa
refactor fetchGlobals and Header
8d0895a
@rccsousa
refactor footer
ad7fd00
@rccsousa
refactor highglights component
e0db3e5 
- update fetchers to include depth parameter to fetch author featured
  images
@rccsousa
fix content filters
96c2342
@rccsousa
finish home page refactor
17d1c9a
@rccsousa
refactor author page
6023ecd
@rccsousa
refactor fetchers
1f84bdc
@rccsousa
conclude author page refactor
df1cbbe
@rccsousa
fix typescript errors for building
a0cde33
@rccsousa
update gitignore
daa70e3
@netlify Netlify
Copy link

netlify bot commented Oct 10, 2024 •
edited
Loading

✅ Deploy Preview for content-sub ready!

Name Link
🔨 Latest commit dc5191c
🔍 Latest deploy log https://app.netlify.com/sites/content-sub/deploys/670952233571160008ffba99
😎 Deploy Preview https://deploy-preview-53--content-sub.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@rccsousa rccsousa changed the title will this deploy? upgrade payload to ver. 3.0 Oct 10, 2024
rccsousa added 20 commits October 11, 2024 11:42
@rccsousa
refactor blogpost page
bf5db92
@rccsousa
fix typescript errors
3384172
@rccsousa
linting
5982cb2
@rccsousa
Merge branch 'main' of https://github.com/subvisual/content-sub into …
1539027 
…rs/upgrade-payload
@rccsousa
fix typescript errors
f08b451
@rccsousa
refactor podcast episode page
fc12b0a
@rccsousa
fix image styling
1f9a5c1
@rccsousa
disable unstyled categories
29de1ef
@rccsousa
modify collection properties
e5770ee 
- remove required featured image
- change roles in Authors to 'type text'
@rccsousa
remove mandatory image field for author
1a9d06a
@rccsousa
fix FeaturedImage conditional logic to avoid rendering errors
331c5bb
@rccsousa
fix blogpost featured image positioning
2d3ce45
@rccsousa
update: add conditional rendering to ListenOn block to only show when…
732cd95 
… available
@rccsousa
update dependencies
80ecde2 
- keep react@rc for payload and use resolutions to have react@latest for
  lexical
@rccsousa
fix react version
b925bf3
@rccsousa
fix package.json
e1eafe4
@rccsousa
update CI linter
baa657d
@rccsousa
fix: remove resolutions
3ccc07d
@rccsousa
revert linter
a0785a2
@rccsousa
Merge branch 'main' of https://github.com/subvisual/content-sub into …
76d13a6 
…rs/upgrade-payload
@rccsousa rccsousa requested a review from davelange October 11, 2024 15:41
@rccsousa rccsousa marked this pull request as ready for review October 11, 2024 15:53
davelange
davelange approved these changes Oct 14, 2024
View reviewed changes
Copy link
Contributor

@davelange davelange left a comment

Choose a reason for hiding this comment

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

👏 👏 👏

@rccsousa rccsousa merged commit ee8fa05 into main Oct 14, 2024
5 checks passed
@rccsousa rccsousa deleted the rs/upgrade-payload branch October 14, 2024 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davelange davelange davelange approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rccsousa @davelange