Skip to content

Comments

Master#1749

Open
xiaowang99199 wants to merge 1 commit intoopenai:masterfrom
xiaowang99199:master
Open

Master#1749
xiaowang99199 wants to merge 1 commit intoopenai:masterfrom
xiaowang99199:master

Conversation

@xiaowang99199
Copy link

  • I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested

Additional context & links

<!-- Thank you for contributing to this project! -->
<!-- The code in this repository is all auto-generated, and is not meant to be edited manually. -->
<!-- We recommend opening an Issue instead, but you are still welcome to open a PR to share for -->
<!-- an improvement if you wish, just note that we are unlikely to merge it as-is. -->

- [ ] I understand that this repository is auto-generated and my pull request may not be merged

## Changes being requested

## Additional context & links
@xiaowang99199 xiaowang99199 requested a review from a team as a code owner February 19, 2026 23:58
Copy link

@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: bb517126fd

ℹ️ 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 on lines +65 to +67
<button
onClick={onBack}
className="px-6 py-2 bg-[#6B7280] hover:bg-[#4B5563] text-white font-medium rounded text-sm transition-colors">

Choose a reason for hiding this comment

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

P1 Badge Submit question instead of navigating back

The primary action button (发送反馈) calls onBack, so clicking it immediately leaves the page instead of submitting the question data. In the current flow, users can fill out the form and lose their input with no save or confirmation, which makes the submit path effectively broken; this handler should trigger a submission action rather than the back navigation callback.

Useful? React with 👍 / 👎.

</button>
<h1 className="text-lg font-bold text-gray-900">新投诉</h1>
</div>
<button className="px-6 py-2 bg-[#6B7280] hover:bg-[#4B5563] text-white font-medium rounded text-sm transition-colors">

Choose a reason for hiding this comment

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

P1 Badge Wire complaint submit button to a submit action

The complaint form's 发送反馈 button is rendered without any click or form submit behavior, so pressing the primary action does nothing. This leaves users unable to complete the complaint workflow after filling required fields, making the form non-functional in production usage.

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.

1 participant