Skip to content

Matching pref rename - #69

Merged
Allimonae merged 1 commit into
mainfrom
matching-pref
Aug 7, 2026
Merged

Matching pref rename#69
Allimonae merged 1 commit into
mainfrom
matching-pref

Conversation

@Allimonae

@Allimonae Allimonae commented Aug 3, 2026

Copy link
Copy Markdown
Member

Renamed the matchingPreferenceOptions variable to matchPrefOptions for brevity and consistency.

Allimonae commented Aug 3, 2026

Copy link
Copy Markdown
Member Author

@Allimonae
Allimonae marked this pull request as ready for review August 3, 2026 19:30
@graphite-app

graphite-app Bot commented Aug 3, 2026

Copy link
Copy Markdown

Graphite Automations

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

2 reviewers were added to this PR based on Henry Chen's automation.

arklian
arklian previously approved these changes Aug 3, 2026
Comment thread js/src/features/sign-up/components/SignUpForm.tsx Outdated
@Allimonae
Allimonae force-pushed the matching-pref branch 2 times, most recently from a2df2e3 to 57ca83c Compare August 6, 2026 19:38
@Allimonae
Allimonae changed the base branch from members-CRUD to graphite-base/69 August 7, 2026 18:17
@graphite-app
graphite-app Bot changed the base branch from graphite-base/69 to main August 7, 2026 18:19
@graphite-app
graphite-app Bot dismissed arklian’s stale review August 7, 2026 18:19

The base branch was changed.

@graphite-app
graphite-app Bot requested a review from a team August 7, 2026 18:19
@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2026

Copy link
Copy Markdown

Allimonae commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

Merge activity

  • Aug 7, 6:44 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Aug 7, 6:44 PM UTC: @Allimonae merged this pull request with Graphite.

@Allimonae
Allimonae merged commit 12b1e59 into main Aug 7, 2026
7 checks passed
@Allimonae
Allimonae deleted the matching-pref branch August 7, 2026 18:44
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