Skip to content

Add help text modals to Cracking and Collapsing fields#1145

Merged
kevinherdez merged 2 commits intomainfrom
claude/issue-962-20260321-1458
Mar 22, 2026
Merged

Add help text modals to Cracking and Collapsing fields#1145
kevinherdez merged 2 commits intomainfrom
claude/issue-962-20260321-1458

Conversation

@rustynwac
Copy link
Contributor

Add help text popup modals to the Cracking and Collapsing fields in the main Observation Form, mimicking the pattern used by avalancheObservationOccurrenceDate in the Avalanche Observation Form.

Closes #962

Generated with Claude Code

…Form

- Add helpText prop support to SwitchField component
- Add observationCracking and observationCollapsing strings to helpStrings.ts
- Wire up help text for instability.cracking and instability.collapsing fields in ObservationForm

Closes #962

Co-authored-by: Rusty <rustynwac@users.noreply.github.com>
@kevinherdez
Copy link
Collaborator

@claude please run yarn prettify and commit the changes

@claude
Copy link

claude bot commented Mar 21, 2026

Claude finished @kevinherdez's task in 2m 52s —— View job


Running yarn prettify

  • Gather context and understand request
  • Install dependencies (yarn install)
  • Run yarn prettify
  • Commit and push changes

Issue Found

I'm unable to run yarn prettify because the workflow's allowedTools has yarn listed as a standalone tool, but yarn commands need to be listed as Bash(yarn*) to be auto-approved. The current config in .github/workflows/claude.yml line 35 has:

--allowedTools "Read,Glob,Grep,Edit,Write,yarn,Bash(pnpm prettify),...

To fix this, the yarn entry should be replaced with Bash(yarn*) (or specific patterns like Bash(yarn prettify), Bash(yarn install)), similar to how the pnpm commands are listed.

For example:

--allowedTools "Read,Glob,Grep,Edit,Write,Bash(yarn*),Bash(git status),Bash(git diff),..."

I'm unable to modify .github/workflows/ files directly (GitHub App permissions don't allow workflow modifications). Please update the workflow file and re-trigger the command.

@kevinherdez kevinherdez added this pull request to the merge queue Mar 22, 2026
Merged via the queue into main with commit 12d9aca Mar 22, 2026
5 checks passed
@kevinherdez kevinherdez deleted the claude/issue-962-20260321-1458 branch March 22, 2026 00:17
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.

Add help text to observation fields

2 participants