- Notifications
You must be signed in to change notification settings - Fork 270
Dockerize #64
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
Dockerize #64
Conversation
| @Feretj is attempting to deploy a commit to the 9d8 Team on Vercel. A member of the Team first needs to authorize it. |
| Caution Review failedThe pull request is closed. WalkthroughThis PR renames environment variables from Changes
Sequence Diagram(s)sequenceDiagram participant Client as Client / Browser participant NextApp as Next.js App participant Docker as Docker Container participant WordPress as WordPress Backend Note over Docker: docker-entrypoint.sh Docker->>Docker: npm run build (SSG + SSR setup) Docker->>NextApp: npm run start Client->>NextApp: Request / NextApp->>NextApp: Home() async function NextApp->>WordPress: getPostsPaginated(1, 30) WordPress-->>NextApp: {posts: [...], headers: {...}} NextApp-->>Client: Render posts grid or "No posts found" Client->>NextApp: Request /posts/[slug] NextApp->>WordPress: getPostById(slug) WordPress-->>NextApp: Post data alt Author available NextApp->>WordPress: getAuthorById(post.author) WordPress-->>NextApp: Author data else Author fetch fails NextApp->>NextApp: author = undefined end NextApp-->>Client: Render post with optional author byline Estimated code review effort🎯 3 (Moderate) | ⏱️ ~30 minutes The changes span multiple layers (config, docs, API, UI, Docker) with a mix of homogeneous updates (env var renames across many files) and heterogeneous logic changes (new pagination API, dynamic rendering, SSG removal). While most individual changes are straightforward, the interconnected nature of pagination logic, architectural shift from static to dynamic generation, and new public API surface in Possibly related PRs
Suggested reviewers
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (20)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
| Sorry made pull request to wrong repository. |
Add Dockerfile and make config changes to include this app in docker-compose
Summary by CodeRabbit
New Features
Chores
Documentation