Skip to content

Feature/editor lazy loading#8

Open
redzzzi wants to merge 5 commits intodevelopfrom
feature/editor-lazy-loading
Open

Feature/editor lazy loading#8
redzzzi wants to merge 5 commits intodevelopfrom
feature/editor-lazy-loading

Conversation

@redzzzi
Copy link
Copy Markdown
Member

@redzzzi redzzzi commented Mar 19, 2026

No description provided.

@cjaewon
Copy link
Copy Markdown
Member

cjaewon commented Mar 19, 2026

@codex review

2 similar comments
@cjaewon
Copy link
Copy Markdown
Member

cjaewon commented Mar 19, 2026

@codex review

@cjaewon
Copy link
Copy Markdown
Member

cjaewon commented Mar 19, 2026

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 72ebc4a2be

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/api/postApi.ts
Comment on lines 169 to 172
default: () => {
// TODO 페이지 문구로 띄워주기
toast.error(BOARD.error.readCondition);
// toast.error(BOARD.error.readCondition);
},
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Restore the read-condition error toast on 400s

This change makes the 400 handler a no-op, so when GET /posts/:id fails because the user has not met the read condition, useGetEachPostQuery now swallows the error completely. BoardView only special-cases 404, so these users are left on an empty page with no toast or inline explanation. Any board that relies on BOARD.error.readCondition will now fail silently.

Useful? React with 👍 / 👎.

Comment thread rsbuild.config.ts
Comment on lines +24 to +27
if (process.env.ANALYZE === 'true') {
chain
.plugin('bundle-analyzer')
.use(BundleAnalyzerPlugin, [{ analyzerMode: 'server' }]);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P3 Badge Use a non-blocking analyzer mode for build:analyze

With analyzerMode: 'server', the new npm run build:analyze command will keep the analyzer HTTP server alive after the build finishes instead of exiting. Rsbuild documents this behavior explicitly, so anyone running the script in automation or expecting a one-shot report will see a hung process. The safer default here is the static report mode that terminates normally.

Useful? React with 👍 / 👎.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants