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

Only the Date field's editor says that reminders are on, so an organizer who looks in Settings finds nothing about them

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

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

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

評価

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

調査の方向性

Start with src/app/app/(chrome)/signups/[id]/settings/page.tsx and read src/lib/reminder-fields.ts for sendsReminders and findReminderFields; compare the wording with InlineFieldForm.tsx and the help page’s Reminders section. Add read-only status for on, off, and no date field, using the reminder field’s label, and update the email line when reminders are on. Add tests for each state and the email wording, then run the listed lint, typecheck, knip, and test commands.

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

説明

enhancement pipeline:proposed pipeline:ux

Needs your OK: this adds a short line about reminders to the Settings tab. Swap pipeline:proposed for pipeline:ready to have it built.

What happens

Reminders are on for every new signup with a Date field. People who gave an email get one the day before their slot. In the app, an organizer learns this in one place only: on the Build tab, under Fields, by editing the Date field. Its "Send a reminder email before this date" checkbox carries the explanation (InlineFieldForm.tsx). The other hints are small:

  • The Fields list marks the reminder field with a bell. Its words, "Reminders sent the day before this date", are only a tooltip and a screen-reader label (FieldsPopover.tsx).
  • The slot editor mentions reminders only once someone has signed up for that slot (SlotEditor.tsx).
  • The help page "Create and publish your first signup" has a Reminders section.

The Settings tab says nothing about reminders while email is required. Its only line about emails is "We email everyone their link to change or cancel." (src/app/app/(chrome)/signups/[id]/settings/page.tsx). In the walkthrough, an organizer building a volunteer shift grid went to Settings and found no mention of reminders.

The switch is where it is on purpose. #200 put it on the date field and removed the Settings tab's Reminders card, so there is one place to change it (see the CHANGELOG, and the comment in the Settings page). That stays. What's missing is a line where organizers look, saying whether reminders are on and where to change them.

Fix, in outline

  • A "Reminders" line on the Settings tab, in the "Sign-up form" card or a small card of its own. It's read-only, with no second switch. It says one of these, with the reminder field's own label in place of "Date":
    • On: "People who give an email get a reminder the day before their slot's Date. To turn reminders off, open Fields on the Build tab and choose Date."
    • Off: "Reminders are off. To turn them on, open Fields on the Build tab and choose Date."
    • No date field: "No reminders: your slots have no Date field."
  • Work it out on the server with sendsReminders(sig.settings) and findReminderFields (src/lib/reminder-fields.ts), from the signup the page already loads. The tab stays a plain server-rendered page with no script of its own.
  • The email line says it too. With reminders on, "We email everyone their link to change or cancel." becomes "We email everyone their link to change or cancel, and a reminder the day before their slot."

Watch for

  • One switch. Don't add a toggle here unless the owner picks that option below.
  • The same words as the builder. Match the field editor's "Sent the day before" and the help page's REMINDER_WHEN, which reads REMINDER_LEAD_HOURS.
  • Email optional. The Optional line already says people without an email get no reminders. Keep it.
  • Help. Add a sentence to the Reminders section of the help page saying the Settings tab shows whether they're on. Open PRs #574, #576, #578, #580 and #581 also change that page. Whichever lands second has a small merge.
  • The CHANGELOG. The legal pages need no change: nothing new is stored or sent.

Choices this makes

  • On the Settings tab, read-only. It's where the walkthrough's organizer looked, and it keeps #200's one switch on the date field. The alternatives:
    • An on/off switch on Settings for sendReminders, with the date field still choosing which date. More direct, but it brings back the card #200 removed.
    • A line on the Build tab, for example beside the Fields button. Closer to the switch, but the Build tab is already busy.
  • The wording above, in plain words as docs/writing-help.md asks.
  • No link that opens the Fields dialog. The text names the steps instead. A link that opens the dialog on the Date field would need a way into the Build tab's state, like #580's BuildBridge.

Done when

  • The Settings tab shows the right line when reminders are on, when they're off, and when the signup has no date field, with the field's own label. A test pins each.
  • With reminders on and email required, the email line mentions the reminder.
  • pnpm lint && pnpm typecheck && pnpm knip && pnpm test pass.

Found by the UX walkthrough (scenario O2) on 2026-10-24, and checked against the code on main at 4ed27a9.

Generated by Claude Code

主要言語
TypeScript
スター
19
フォーク
4
平均マージ
19時間 34分
マージ済み PR（30日）
165

環境構築

はじめの一歩

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

richshaw/OpenSignup のほかの issue

richshaw/OpenSignup の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

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

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