-
Notifications
You must be signed in to change notification settings - Fork 4.7k
docs: update next-ssr tasks #1883
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
base: master
Are you sure you want to change the base?
Conversation
| ### Feature 1: Application Continuity (**10 points**) | ||
|
|
||
| **As a** developer | ||
| **I want** the application to function according to previous modules' requirements |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's explicitly mention exclusion of the Forms module
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
|
||
| ### Feature 1: Application Continuity (**10 points**) | ||
|
|
||
| **As a** developer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's add a list of the requirements which one need to implement using justr Next.js (wuthout migrating).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated here, 1aa20c5
let me know if i've missed anything
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey!
I'm sorry for a confusion. The main idea was that we don't want students to migrate the app, let them write a new one using NextJS cli instead.
|
|
||
| - All links use next-intl createNavigation. [10 points] | ||
|
|
||
| ### Feature 7: Server-Side Static Requests (**20 points**) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Provide specific requirements about functionality
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
|
||
| - Search and filter submissions are wired through server actions or server functions, not ad-hoc client-only fetch calls. [5 points] | ||
| - Selecting an item for the details panel triggers server-side data fetching for that item (via server component or server function). [5 points] | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cross-check (score can be less if some parts of the functionality don't work)
Could you please delete score below because these points added in the Acceptance Criteria above.
Title of Pull Request
Restructure next.js ssr tasks
🤔 This is a
Description
Additional Information
https://github.com/rolling-scopes-school/tasks/blob/feature/ssr-tasks/react/modules/tasks/nextjs-ssr-ssg.md
Checklist