Skip to content

MemberService.java updateMember - #68

Merged
Allimonae merged 2 commits into
mainfrom
members-CRUD
Aug 7, 2026
Merged

MemberService.java updateMember#68
Allimonae merged 2 commits into
mainfrom
members-CRUD

Conversation

@Allimonae

@Allimonae Allimonae commented Aug 3, 2026

Copy link
Copy Markdown
Member

MemberService updateMember Implemented

The updateMember method now performs a true partial update instead of replacing the entire member object. Only fields provided in the UpdateMemberRequest that are non-null will be applied to the existing member, preserving all other field values.

@NotBlank has been removed from fullName, email, and introduction in UpdateMemberRequest to allow partial updates.

Test Coverage

  • Throws MemberNotFoundException when the member does not exist
  • Updates only the provided field (fullName) while preserving all other existing values
  • Leaves all fields unchanged when all request fields are null
  • Updates all fields when all request fields are provided

Allimonae commented Aug 3, 2026

Copy link
Copy Markdown
Member Author

@Allimonae Allimonae changed the title Instant to OffsetDateTime MemberService.java updateMember Aug 3, 2026
@Allimonae
Allimonae marked this pull request as ready for review August 3, 2026 19:19
@Allimonae
Allimonae requested a review from a team August 3, 2026 19:19
@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.

@Allimonae Allimonae mentioned this pull request Aug 3, 2026

Allimonae commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

Merge activity

  • Aug 7, 6:16 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Aug 7, 6:17 PM UTC: Graphite rebased this pull request as part of a merge.
  • Aug 7, 6:18 PM UTC: @Allimonae merged this pull request with Graphite.

@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2026

Copy link
Copy Markdown

@Allimonae
Allimonae merged commit d7d7a6b into main Aug 7, 2026
7 checks passed
@Allimonae
Allimonae deleted the members-CRUD branch August 7, 2026 18:18
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.

3 participants