イベントの主催の変更後にも以前の主催に#services/knoQで通知が飛ぶ #404

sapphi-red · 2023-05-29T08:40:55Z

https://q.trap.jp/messages/9b602aa3-d266-4f09-bba6-c4c226b4bc06

Luftalian · 2023-07-19T01:50:31Z

Lines 56 to 67 in cb9fc32

    
           for _, groupMember := range group.Members { 
        
           	exist := false 
        
           	for _, currentAttendee := range currentEvent.Attendees { 
        
           		if currentAttendee.UserID == groupMember.ID { 
        
           			exist = true 
        
           		} 
        
           	} 
        
           	if !exist { 
        
           		_ = repo.GormRepo.UpsertEventSchedule(event.ID, groupMember.ID, domain.Pending) 
        
           	} 
        
           }

knoQ/router/middleware.go

Lines 218 to 225 in cb9fc32

    
           for _, attendee := range e.Attendees { 
        
           	if attendee.Schedule == presentation.Pending { 
        
           		user, ok := usersMap[attendee.ID] 
        
           		if ok { 
        
           			nofiticationTargets = append(nofiticationTargets, user.Name) 
        
           		} 
        
           	} 
        
           }

主催が変わったとき、変更前の主催のメンバーが変更後の主催に含まれない場合、そのメンバーは出席状況が未定に戻される。
Webhookは出席状況が未定の全てのユーザーにメンションが飛ぶ

上のような状況になっているため、このIssueが起きているようです。

Luftalian · 2023-07-19T01:51:17Z

変更案としては、

Webhookの通知を飛ばすのを出席状況が未定の全てのユーザーではなく、Groupのメンバーの中で出席状況が未定のユーザーのみにする。
主催が変わったとき、変更前の主催のメンバーが変更後の主催に含まれない場合、そのメンバーをイベントのメンバーから削除する

が考えられそうです。
@ras0q どっちのほうがいいですか？

ras0q · 2023-07-20T05:12:44Z

調査ありがとうございます！

個人的には2がよさそうなんですが、新グループに含まれないかつ予定を記入してないことを条件にすると足りそうなきがします

Luftalian · 2023-07-20T13:19:02Z

新グループに含まれないかつ予定を記入してないことメンバーのみ削除するということですか？

Luftalian · 2023-07-20T13:20:34Z

グループ外からの参加ができないイベントの場合は、新グループに含まれないメンバーは問答無用で削除でいいですか？

ras0q · 2023-07-24T11:20:05Z

この仕様忘れてました、これでOKです

グループ外からの参加ができないイベントの場合は、新グループに含まれないメンバーは問答無用で削除でいいですか？

グループ外からもOKな場合はこれでお願いします:pray:

新グループに含まれないかつ予定を記入してないことメンバーのみ削除するということですか？

ras0q · 2023-07-24T11:21:15Z

グループ外NGのときorグループ外NGに変更するときはUIからwarning出るようにしたいかも
フロントエンドの方にissue建てておきます

Luftalian · 2023-08-02T11:41:49Z

変更方針

前の主催チームのメンバーが後の主催チームのメンバーでもあった場合、出席状況はそのまま追加する
前の主催チームのメンバーが後の主催チームのメンバーでなかった場合、出席状況は以下の通りにする

チーム外参加不可の場合、問答無用で出席状況を削除する。(Pendingではない)
{チーム外参加可}かつ{前の主催チームのメンバーの出席状況が出席Attendance}の場合はそのまま追加する。
{チーム外参加可}かつ{前の主催チームのメンバーの出席状況が欠席Absent}の場合はそのまま追加する。~~(or 出席状況を削除する。(Pendingではない))???~~
{チーム外参加可}かつ{前の主催チームのメンバーの出席状況が未定Pending}の場合は出席状況を削除する。(Pendingではない)

現状

前の主催チームのメンバーが後の主催チームのメンバーでもあった場合、出席状況はそのまま追加する
前の主催チームのメンバーが後の主催チームのメンバーでなかった場合、前の主催チームのメンバーの出席状況を全てPendingにしている

現状の問題の原因

WebhookではPendingのメンバー全てにメンションするため、出席状況を書いていなかった前の主催チームのメンバーには必ずメンションが飛ぶ

ras0q · 2023-08-02T12:08:00Z

        for _, groupMember := range group.Members {
                exist := false
+               schedule := domain.Pending
                for _, currentAttendee := range currentEvent.Attendees {
                        if currentAttendee.UserID == groupMember.ID {
                                exist = true
                        }
+                       schedule = currentAttendee.Schedule
                }
-               if !exist {
-                       _ = repo.GormRepo.UpsertEventSchedule(event.ID, groupMember.ID, domain.Pending)
+               if !exist && schedule == domain.Pending {
+                       _ = repo.GormRepo.DeleteMemberOfGroup(eventID, groupMember.ID)
                }
 
        }

ras0q · 2023-08-02T12:09:50Z

出席または欠席にしていたらそのまま残す
何も入力していないorPending(内部ではどちらもPendingとして扱われている)の場合は削除

ras0q · 2023-08-02T12:12:37Z

DeleteMemberOfGroupは別のメソッドなので別途作る必要がある

ras0q · 2023-08-02T12:52:41Z

	attendeeMap := make(map[uuid.UUID]domain.ScheduleStatus)
	for _, attendee := range currentEvent.Attendees {
		attendeeMap[attendee.UserID] = attendee.Schedule
	}

	for _, groupMember := range group.Members {
		_, ok := attendeeMap[groupMember.ID]
		if !ok {
			_ = repo.GormRepo.UpsertEventSchedule(event.ID, groupMember.ID, domain.Pending)
		}
	}

	for aid, schedule := range attendeeMap {
		ok := slices.ContainsFunc(group.Members, func(m domain.User) bool { return m.ID == aid })
		if !ok && schedule == domain.Pending {
			_ = repo.GormRepo.DeleteEventSchedule(event.ID, aid)
		}
	}

Luftalian · 2023-08-02T13:16:12Z

event.AllowTogetherを考慮した

attendeeMap := make(map[uuid.UUID]domain.ScheduleStatus)
for _, attendee := range currentEvent.Attendees {
	attendeeMap[attendee.UserID] = attendee.Schedule
}

for _, groupMember := range group.Members {
	_, ok := attendeeMap[groupMember.ID]
	if !ok {
		_ = repo.GormRepo.UpsertEventSchedule(event.ID, groupMember.ID, domain.Pending)
	}
}

for aid, schedule := range attendeeMap {
	ok := slices.ContainsFunc(group.Members, func(m domain.User) bool { return m.ID == aid })
	if !ok && (!event.AllowTogether || schedule == domain.Pending) {
		_ = repo.GormRepo.DeleteEventSchedule(event.ID, aid)
	}
}

Luftalian · 2023-08-13T04:43:54Z

Luftalian · 2023-08-13T05:12:52Z

8個ぐらい試してみないといけないパターンがあると思っているんですが、ローカル環境でどうやって管理者外の場合とかを試せばいいかわからないです。

ras0q · 2023-08-14T08:03:25Z

手で直接確かめるのは2人必要なのでなかなか難しそうですね
UpdateEventのテストを書けると良さそう
詰まったら教えてください

Luftalian · 2023-08-15T05:50:04Z

テストのためにproductionパッケージのRepository構造体を用意するところでGormRepo db.GormRepositoryをどうやって作ればいいのかわかりません。

ras0q · 2023-08-15T06:34:16Z

https://github.com/DATA-DOG/go-sqlmock を使ってモックを書くか https://github.com/ory/dockertest とかで一時的にテスト用のDBを立てると良いと思います
どっちもtraPortfolioに書いてあるので参考にしてもらえると

Luftalian · 2023-08-28T13:19:54Z

テストは書くのが難しそうなのでテスト作成は保留となった

sapphi-red added the bug Something isn't working label May 29, 2023

Luftalian self-assigned this Jul 18, 2023

ras0q mentioned this issue Aug 2, 2023

グループ外NGのときorグループ外NGに変更するときはUIからwarning出るようにしたい traPtitech/knoQ-UI#893

Closed

Luftalian linked a pull request Aug 28, 2023 that will close this issue

UpdateEventの動作を変更する #447

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

イベントの主催の変更後にも以前の主催に#services/knoQで通知が飛ぶ #404

イベントの主催の変更後にも以前の主催に#services/knoQで通知が飛ぶ #404

sapphi-red commented May 29, 2023

Luftalian commented Jul 19, 2023

Luftalian commented Jul 19, 2023 •

edited

Loading

ras0q commented Jul 20, 2023

Luftalian commented Jul 20, 2023

Luftalian commented Jul 20, 2023

ras0q commented Jul 24, 2023

ras0q commented Jul 24, 2023 •

edited

Loading

Luftalian commented Aug 2, 2023 •

edited

Loading

ras0q commented Aug 2, 2023 •

edited

Loading

ras0q commented Aug 2, 2023

ras0q commented Aug 2, 2023

ras0q commented Aug 2, 2023 •

edited

Loading

Luftalian commented Aug 2, 2023 •

edited

Loading

Luftalian commented Aug 13, 2023

Luftalian commented Aug 13, 2023

ras0q commented Aug 14, 2023

Luftalian commented Aug 15, 2023

ras0q commented Aug 15, 2023

Luftalian commented Aug 28, 2023

イベントの主催の変更後にも以前の主催に#services/knoQで通知が飛ぶ #404

イベントの主催の変更後にも以前の主催に#services/knoQで通知が飛ぶ #404

Comments

sapphi-red commented May 29, 2023

Luftalian commented Jul 19, 2023

Luftalian commented Jul 19, 2023 • edited Loading

ras0q commented Jul 20, 2023

Luftalian commented Jul 20, 2023

Luftalian commented Jul 20, 2023

ras0q commented Jul 24, 2023

ras0q commented Jul 24, 2023 • edited Loading

Luftalian commented Aug 2, 2023 • edited Loading

変更方針

現状

現状の問題の原因

ras0q commented Aug 2, 2023 • edited Loading

ras0q commented Aug 2, 2023

ras0q commented Aug 2, 2023

ras0q commented Aug 2, 2023 • edited Loading

Luftalian commented Aug 2, 2023 • edited Loading

Luftalian commented Aug 13, 2023

Luftalian commented Aug 13, 2023

ras0q commented Aug 14, 2023

Luftalian commented Aug 15, 2023

ras0q commented Aug 15, 2023

Luftalian commented Aug 28, 2023

Luftalian commented Jul 19, 2023 •

edited

Loading

ras0q commented Jul 24, 2023 •

edited

Loading

Luftalian commented Aug 2, 2023 •

edited

Loading

ras0q commented Aug 2, 2023 •

edited

Loading

ras0q commented Aug 2, 2023 •

edited

Loading

Luftalian commented Aug 2, 2023 •

edited

Loading