These are chat archives for fossasia/open-event-server

10th
Jun 2018
Kreijstal
@Kreijstal
Jun 10 2018 08:36
Why is OrderStatisticsEventDetail not an EventRelationship?
Bhavesh Anand
@bhaveshAn
Jun 10 2018 14:29
@Kreijstal @Masquerade0097 @mayank8318 @maxlorenz @schedutron Please review fossasia/open-event-server#4872 fossasia/open-event-server#4724 fossasia/open-event-server#4836
Kreijstal
@Kreijstal
Jun 10 2018 18:46
Maybe you should try smaller pull requests
When you're PRs are long they will make it very hard to review
Bhavesh Anand
@bhaveshAn
Jun 10 2018 20:50
Okay!

@niranjan94 @iamareebjamal I have made changes as were suggested. Please review fossasia/open-event-server#4724

screenshot from 2018-06-09 18-25-23
screenshot from 2018-06-09 18-25-15

Mario Behling
@mariobehling
Jun 10 2018 22:20
@srv-twry Ok, thanks. I gave you additional rights, but please follow best practices. Do not merge your own PRs. Ask others to merge it for you.
Saurav Tiwary
@srv-twry
Jun 10 2018 22:21
Yes absolutely. Thanks @mariobehling