search

langrenn-sprint / race-service

Service to administer racelist and race plan
Apache License 2.0
0 stars 0 forks source link

get_time_events filter on bib#168 #169

Closed heming-langrenn closed 1 year ago

heming-langrenn commented 1 year ago

168

heming-langrenn commented 1 year ago

Hei, @stigbd - kan du gjøre review på #168?

stigbd commented 1 year ago

Det ser ut til at du kun vil merge inn ein ny test. Er alt med i denne PR som skal vere med?

heming-langrenn commented 1 year ago

Ja. Men du må se på #168. Der er det en endring som hører til som glapp rett ut på mail branch.

Sendt fra min iPad

  1. mar. 2023 kl. 23:31 skrev Stig B. Dørmænen @.***>:



Det ser ut til at du kun vil merge inn ein ny test. Er alt med i denne PR som skal vere med?

— Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Flangrenn-sprint%2Frace-service%2Fpull%2F169%23issuecomment-1489355284&data=05%7C01%7C%7C3bca68f417384c93371c08db309cea1c%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C638157222715492020%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=SKMrEScMWC2%2FF7N7pM248L2jJTfxWKcVfbRj2Qm5jsc%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANOXGMYZGLHSX2G5ADXJ2U3W6SSZ3ANCNFSM6AAAAAAWMMVZ64&data=05%7C01%7C%7C3bca68f417384c93371c08db309cea1c%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C638157222715492020%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=3eIoPJ102LzWShNklPd2u2fWEbkCWPTFTAuLpPiuj%2BQ%3D&reserved=0. You are receiving this because you authored the thread.Message ID: @.***>

stigbd commented 1 year ago

Eg antar at du kun vil ha time_events for gitt event og bib, ikkje alle bibs på tvers av alle eventer.

I så fall har eg pusha ein commit som realiserer dette på ein effektiv måte. Du kan jo vurdere den, og reverte om du ikkje synes om den. Sjå 934e13f

heming-langrenn commented 1 year ago

Yes, supert!30. mar. 2023 09:55 skrev "Stig B. Dørmænen" @.***>: Eg antar at du kun vil ha time_events for gitt event og bib, ikkje alle bibs på tvers av alle eventer. I så fall har eg pusha ein commit som realiserer dette på ein effektiv måte. Du kan jo vurdere den, og reverte om du ikkje synes om den. Sjå 934e13f

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>