Skip to content

add series instruments to manage group page#1325

Merged
joshunrau merged 3 commits intoDouglasNeuroInformatics:mainfrom
joshunrau:devel
Mar 30, 2026
Merged

add series instruments to manage group page#1325
joshunrau merged 3 commits intoDouglasNeuroInformatics:mainfrom
joshunrau:devel

Conversation

@joshunrau
Copy link
Copy Markdown
Collaborator

@joshunrau joshunrau commented Mar 30, 2026

Summary by CodeRabbit

  • New Features

    • Added ability to select and manage series instruments when configuring group access.
    • Added multilingual support (English/French) for series selection in group management.
  • Chores

    • Bumped version to 1.13.5.
    • Updated package dependency configuration.

@joshunrau joshunrau merged commit ac960a1 into DouglasNeuroInformatics:main Mar 30, 2026
1 check passed
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 30, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 58d00a2a-164f-42b2-8133-7c6fbe9b6f3c

📥 Commits

Reviewing files that changed from the base of the PR and between 7b90b70 and 7bddaeb.

📒 Files selected for processing (3)
  • apps/web/src/routes/_app/group/manage.tsx
  • apps/web/src/translations/group.json
  • package.json

Walkthrough

Added series instrument selection support to the group management form with validation and translations, including version bump to 1.13.5 and pnpm configuration updates for dependency handling.

Changes

Cohort / File(s) Summary
Form Enhancement
apps/web/src/routes/_app/group/manage.tsx
Added accessibleSeriesInstrumentIds field to ManageGroupForm with multi-select listbox UI, Zod validation, and form submission handling for series instrument selections.
Translations
apps/web/src/translations/group.json
Added localized "Series" label entries for English and French under manage.series key.
Package Configuration
package.json
Bumped version to 1.13.5; updated pnpm onlyBuiltDependencies to include cypress and sqlite3; added pnpm ignoredBuiltDependencies for @scarf/scarf.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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