Skip to content
This repository has been archived by the owner on Dec 11, 2024. It is now read-only.

feat: replyToMessageの配列やめた #77

Merged
merged 3 commits into from
Apr 24, 2024
Merged

Conversation

Daaaai0809
Copy link
Contributor

No description provided.

Copy link

cloudflare-workers-and-pages bot commented Apr 24, 2024

Deploying meline with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1d5ea24
Status: ✅  Deploy successful!
Preview URL: https://aa6cce2e.meline.pages.dev
Branch Preview URL: https://feat-refactor-replytomessage.meline.pages.dev

View logs

@Daaaai0809 Daaaai0809 self-assigned this Apr 24, 2024
@Daaaai0809 Daaaai0809 requested a review from sor4chi April 24, 2024 13:50
@Daaaai0809 Daaaai0809 force-pushed the feat/refactor-replyToMessage branch from 70db377 to 2dbd037 Compare April 24, 2024 14:27
@Daaaai0809 Daaaai0809 changed the title feat: replyToMessageでidだけ返すように修正 feat: replyToMessageの配列やめた Apr 24, 2024
@Daaaai0809 Daaaai0809 force-pushed the feat/refactor-replyToMessage branch from f02ff4f to dd184a2 Compare April 24, 2024 17:27
@Daaaai0809 Daaaai0809 requested a review from monica-0218 April 24, 2024 17:30
Copy link
Member

@sor4chi sor4chi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nillableかどうかをentity内で表現しましょう

server/domain/entity/message.go Outdated Show resolved Hide resolved
server/domain/entity/message.go Outdated Show resolved Hide resolved
server/domain/entity/message.go Show resolved Hide resolved
server/domain/entity/message.go Outdated Show resolved Hide resolved
@Daaaai0809 Daaaai0809 requested a review from sor4chi April 24, 2024 17:58
@sor4chi sor4chi merged commit dfbf439 into main Apr 24, 2024
5 checks passed
@sor4chi sor4chi deleted the feat/refactor-replyToMessage branch April 24, 2024 19:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants