Hacktoberfest 2026：メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

[BUG]: "Delete" PAT button is gated on the input field, so a saved token can't be removed when the field is empty

オープン 初心者向け
  • 何か問題がありますか？
#401 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 3 日以内に返信

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
88/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
javascript, react
領域
authentication, frontend

調査の方向性

src/pages/SettingsPage.jsx の205行目付近と src/components/PATModal.jsx の227行目付近にある Delete ボタンのロジックから確認します。保存済みの PAT と入力中の下書きがどのように表現されているかを確認し、入力欄が空でも保存済みトークンを削除できるよう、無効化条件を更新します。両方のビューで、削除によって保存済みトークンが取り除かれ、認証状態がクリアされることを確認します。

索引モデルが issue の本文から書いたものです。

説明

bug
Bug Description

In Settings (src/pages/SettingsPage.jsx line 205) and the PAT modal
(src/components/PATModal.jsx line 227), the Delete button is disabled={!draft.trim()} —
gated on the text currently in the input, not on whether a token is actually saved (pat).

So a signed-in user (green AUTHENTICATED pill, token in localStorage) who clears the input
box can no longer revoke their stored token: Delete is disabled. The button also has no
disabled styling, so it still looks clickable but does nothing.

Verified in the running app after clearing the field:
{ inputValue:"", deleteDisabled:true, authenticatedPillShown:true, patStillInStorage:"ghp_..." }
then clicking Delete is a no-op:
{ patStillInStorage:"ghp_...", stillAuthenticated:true }

Steps to Reproduce
  1. Settings: save a PAT (the AUTHENTICATED pill appears).
  2. Select all in the token field and delete the text.
  3. The Delete button is now disabled — you can't remove the stored token without retyping.
Logs and Screenshots

After clearing the input field (token still saved):
inputValue : ""
deleteDisabled : true
authenticatedPillShown : true
patStillInStorage : "ghp_..."

After clicking Delete (no-op — button is disabled):
patStillInStorage : "ghp_..."
stillAuthenticated : true
📎 Image file:

Image
Environment Details
  • OS: macOS - Browser: Chrome
  • Repo: main @ d91d2b8
  • Files: src/pages/SettingsPage.jsx (line 205), src/components/PATModal.jsx (line 227)
  • Suggested fix: gate Delete on pat (token exists), e.g. disabled={!pat || isValidating};
    same change in PATModal.jsx. Related but closed: #321, #138, #257.
Impact

Low - Minor inconvenience

Code of Conduct
  • I have joined the Discord server and will post updates there
  • I have searched existing issues to avoid duplicates
主要言語
JavaScript
スター
45
フォーク
148
平均マージ
5日 20時間
マージ済み PR（30日）
27

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

AOSSIE-Org/OrgExplorer のほかの issue

AOSSIE-Org/OrgExplorer の issue をすべて見る

似ている issue

JavaScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。