[01:31:54] <grrrit-wm>	 (PS1) Mwalker: Make YTD Values Actually YTD [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/101168
[01:43:41] <grrrit-wm>	 (CR) Adamw: Make YTD Values Actually YTD (2 comments) [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/101168 (owner: Mwalker)
[01:46:21] <grrrit-wm>	 (CR) Mwalker: Make YTD Values Actually YTD (2 comments) [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/101168 (owner: Mwalker)
[01:50:52] <grrrit-wm>	 (CR) Adamw: Make YTD Values Actually YTD (1 comment) [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/101168 (owner: Mwalker)
[01:51:11] <grrrit-wm>	 (PS2) Adamw: WIP (FR #917) Convert dangerous queries to use the Civi db connection [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/100696
[01:54:39] <grrrit-wm>	 (PS1) Mwalker: Filter out non online donations [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/101172
[02:03:46] <grrrit-wm>	 (CR) Adamw: [C: 2] Make YTD Values Actually YTD [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/101168 (owner: Mwalker)
[02:03:51] <grrrit-wm>	 (CR) Adamw: [C: 2] Filter out non online donations [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/101172 (owner: Mwalker)
[18:12:24] <grrrit-wm>	 (PS3) Adamw: WIP (FR #917) Convert dangerous queries to use the Civi db connection [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/100696
[18:14:01] <grrrit-wm>	 (PS4) Adamw: WIP (FR #917) Convert dangerous queries to use the Civi db connection [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/100696
[18:14:02] <grrrit-wm>	 (PS1) Adamw: Update and add to tests [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/101237
[18:27:02] <grrrit-wm>	 (PS1) Adamw: Fix bad calls in dcbfae0f [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/101240
[18:28:17] <grrrit-wm>	 (PS1) Adamw: Protect schema upgrades [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/101241
[18:30:10] <awight>	 K4-713: have two seconds for that one ^^ ?
[18:30:36] <awight>	 I'm just... sad when I try to merge multiple commits to mark dependencies in gerrit...
[18:30:49] <awight>	 So it would be nice to get that onto master.
[18:31:38] <K4-713>	 awight: Which one? There are, like, five up there.
[18:31:50] <awight>	 https://gerrit.wikimedia.org/r/101241
[18:31:53] <awight>	 is sort of the blocker.
[18:31:58] <K4-713>	 looking
[18:32:01] <awight>	 lack thereof is killing my unit testing.
[18:33:29] <grrrit-wm>	 (CR) Katie Horn: [C: 2 V: 2] Protect schema upgrades [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/101241 (owner: Adamw)
[18:33:43] <awight>	 woot
[18:50:36] <grrrit-wm>	 (PS2) Adamw: Update and add to tests [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/101237
[19:04:56] <grrrit-wm>	 (PS1) Mwalker: Adding already existing samarium files [wikimedia/fundraising/DataDashboard] - https://gerrit.wikimedia.org/r/101246
[19:09:03] <grrrit-wm>	 (CR) Ssmith: [C: 2 V: 2] Adding already existing samarium files [wikimedia/fundraising/DataDashboard] - https://gerrit.wikimedia.org/r/101246 (owner: Mwalker)
[19:11:13] <slander>	 #984: (AW) Description changed -- https://wikimedia.mingle.thoughtworks.com/projects/online_fundraiser/cards/984
[20:16:03] <grrrit-wm>	 (PS2) Adamw: (FR #1093) Support queue item download [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/100419
[20:52:42] <grrrit-wm>	 (PS2) Adamw: Fix bad calls in dcbfae0f [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/101240
[22:52:12] <slander>	 #751: (AW) Description changed -- https://wikimedia.mingle.thoughtworks.com/projects/online_fundraiser/cards/751
[22:52:27] <awight>	 K4-713: ^^ U agree with the duplicate strategy, or would you rather see it do the WR1 thing?
[22:52:55] <awight>	 Personally, I like to keep civi interfacing encapsulated in the queue consumers.
[22:53:11] <slander>	 #751: (AW) ->Sprint 36 -- https://wikimedia.mingle.thoughtworks.com/projects/online_fundraiser/cards/751
[22:53:15] <K4-713>	 Ehh...
[22:53:23] <K4-713>	 So, I have a couple thoughts about this.
[22:53:31] <K4-713>	 And, there seems to be one new clear answer.
[22:53:52] <K4-713>	 If the point is "take the load off prod", we should be looking to see if the transactions exist... on the read-only.
[22:54:05] <awight>	 good point.
[22:54:33] * K4-713  thinks more

[22:54:42] * K4-713  realizes it's Friday

[22:56:12] <slander>	 #751: (AW) Description changed -- https://wikimedia.mingle.thoughtworks.com/projects/online_fundraiser/cards/751
[23:55:10] <grrrit-wm>	 (PS1) Mwalker: I think I fixed the unsubscribe bugs... [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/101455