Skip to content

Conversation

@IlinJoy
Copy link
Owner

@IlinJoy IlinJoy commented Aug 17, 2025

  1. Task: link
  2. Screenshot:
image
  1. Deploy: link
  2. Done 18.08.2025 / deadline 18.08.2025
  3. Score: 90 / 100
  • Next.js App Router is used in "nextjs-ssr" branch. Pages render on the server with RSC - 20
  • The application supports at least two languages - 10
  • Shared layout is used at the root of application - 5
  • All images are rendered via the next/image component. If your API doesn't support images, then you receive 10 points automatically. - 10
  • All links are rendered via the next-intl createNavigation function - 10
  • 404 error page implemented - 5
  • All static requests are done via the server - 20
  • CSV file compiled in the server action - 20

export const LazyThemeContextProvider = dynamic(
() => import('@/context/theme/ThemeContext').then((module) => module.ThemeContextProvider),
{
ssr: false,
Copy link
Collaborator

Choose a reason for hiding this comment

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

А это нужно, потому что компонент клиентский?
Просто локально тогда такая фигня
image

Copy link
Collaborator

Choose a reason for hiding this comment

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

За такое type safety респект!

Copy link
Collaborator

Choose a reason for hiding this comment

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

А для некста плагин стоит?

Comment on lines +31 to +33
if (response.statusText) {
await new Promise((res) => setTimeout(res, 400));
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

А это для чего нужно?

@@ -0,0 +1,5 @@
import { About } from '@/components/About/About';

export default function aboutPage() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

AboutPage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants