Skip to content

Refine CSV import interaction and submission feedback - #108

Merged
Arshadul-Monir merged 9 commits into
mainfrom
bulk-import-frontend
Oct 26, 2026
Merged

Arshadul-Monir merged 9 commits into
mainfrom
bulk-import-frontend

Conversation

@Arshadul-Monir

@Arshadul-Monir Arshadul-Monir commented Sep 18, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Add drag and drop to the member CSV uploader.
  • Summarize bulk import successes in one notification and retain per-member error feedback.
  • Update member page tests for the revised notifications.

Stack

3 of 3. Depends on #110, which depends on #109.

Verification

  • TypeScript type check passed at this branch tip.
  • All 15 Members page tests passed at this branch tip.

Images

Manual Entry Modal

image.png

Bulk CSV Upload Modal

image

Successful Upload

image.png

Additional Headers

image

Uploading Duplicate User

image

Invalid Column Order

image

@Arshadul-Monir
Arshadul-Monir requested a review from a team September 18, 2026 01:20
@graphite-app
graphite-app Bot requested a review from arklian September 18, 2026 01:21
@graphite-app

graphite-app Bot commented Sep 18, 2026

Copy link
Copy Markdown

Graphite Automations

"Request reviewers once CI passes" took an action on this PR • (09/18/26)

1 reviewer was added to this PR based on Henry Chen's automation.

@Arshadul-Monir Arshadul-Monir changed the title Bulk import frontend Refine CSV import interaction and submission feedback Sep 18, 2026
@Arshadul-Monir
Arshadul-Monir changed the base branch from main to bulk-import-member-form September 18, 2026 01:32

Arshadul-Monir commented Sep 18, 2026 •

Copy link
Copy Markdown
Member Author

@Arshadul-Monir
Arshadul-Monir force-pushed the bulk-import-member-form branch 2 times, most recently from 5b3bc9f to ff8ec84 Compare September 21, 2026 18:35
@Arshadul-Monir
Arshadul-Monir force-pushed the bulk-import-member-form branch from ff8ec84 to 95237df Compare October 9, 2026 22:38
@Arshadul-Monir
Arshadul-Monir force-pushed the bulk-import-frontend branch 2 times, most recently from 40566cc to fdf802f Compare October 9, 2026 22:39
@Arshadul-Monir
Arshadul-Monir force-pushed the bulk-import-member-form branch from 95237df to f18a5fb Compare October 9, 2026 22:53
Comment thread js/src/features/members/Members.page.test.tsx Outdated
@Arshadul-Monir
Arshadul-Monir force-pushed the bulk-import-member-form branch from f18a5fb to 6b5f5fc Compare October 14, 2026 20:50
@Arshadul-Monir
Arshadul-Monir force-pushed the bulk-import-frontend branch 4 times, most recently from fedf994 to a6c5fa8 Compare October 16, 2026 15:29
Comment thread js/src/features/members/components/AddMembersModal.tsx
@Arshadul-Monir
Arshadul-Monir force-pushed the bulk-import-member-form branch from 6b5f5fc to d508ae7 Compare October 16, 2026 22:02
@Arshadul-Monir
Arshadul-Monir force-pushed the bulk-import-frontend branch 2 times, most recently from 146c822 to 1e7469f Compare October 22, 2026 02:08
@Arshadul-Monir
Arshadul-Monir force-pushed the bulk-import-member-form branch from d508ae7 to 56b0762 Compare October 22, 2026 02:08
@spiffyy99
spiffyy99 self-requested a review October 22, 2026 16:41
spiffyy99
spiffyy99 previously approved these changes Oct 22, 2026
Arshadul-Monir added a commit that referenced this pull request Oct 22, 2026
## Summary
- Parse member CSV files, validate required headers and row values, and flag duplicate emails.
- Add a CSV file picker with a preview of parsed rows and validation errors.

## Stack
1 of 3. #110 connects this component to the Members page; #108 adds interaction and feedback refinements.

## Verification
- TypeScript type check passed at this branch tip.
@Arshadul-Monir
Arshadul-Monir force-pushed the bulk-import-member-form branch 2 times, most recently from 04b1f2a to 4fdb3aa Compare October 22, 2026 20:40
@Arshadul-Monir
Arshadul-Monir changed the base branch from bulk-import-member-form to graphite-base/108 October 22, 2026 20:49
Arshadul-Monir added a commit that referenced this pull request Oct 22, 2026
## Summary
- Add a modal for manual member entry and CSV import, with validation and submission feedback.
- Connect the modal to the Members page.
- Test manual submission, CSV import, API errors, and existing page behavior.

## Stack
2 of 3. Depends on #109. #108 is the final stage.

## Verification
- TypeScript type check passed at this branch tip.
- All 14 Members page tests passed at this branch tip.
@graphite-app
graphite-app Bot changed the base branch from graphite-base/108 to main October 22, 2026 20:50
@graphite-app
graphite-app Bot dismissed spiffyy99’s stale review October 22, 2026 20:50

The base branch was changed.

@sonarqubecloud

Copy link
Copy Markdown

@Arshadul-Monir
Arshadul-Monir requested review from spiffyy99 and removed request for arklian October 26, 2026 16:12

Arshadul-Monir commented Oct 26, 2026 •

Copy link
Copy Markdown
Member Author

Merge activity

  • Oct 26, 4:26 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Oct 26, 4:26 PM UTC: @Arshadul-Monir merged this pull request with Graphite.

@Arshadul-Monir
Arshadul-Monir merged commit b34cffa into main Oct 26, 2026
7 checks passed
@Arshadul-Monir
Arshadul-Monir deleted the bulk-import-frontend branch October 26, 2026 16:26
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.

2 participants