Wed, Oct 23
Thu, Oct 17
Wed, Oct 16
@jijiki Let's go with sustainability@. Thanks!
Oct 10 2019
Good question @Aklapper! It is environmental sustainability. I was going for brevity, but I'm open to firstname.lastname@example.org if it makes more sense.
Oct 8 2019
Basically the issue seems to be that not opting_in doesn't opt out of bulk mail, correct @MBeat? Did the donor get a bulk email? Also is this the behavior we want @CCogdill_WMF?
@MBeat it looks like the contact was created on 10/2 and the only change after that was the activity record.
@MBeat Thanks! I see opt-in listed as no, and my expectation is that that's what that form maps to, but do you think she would also get "On Hold" on her email address? I don't see "is_opt_out" available on her record as this documentation would suggest. https://www.mediawiki.org/wiki/Fundraising_tech/CiviCRM Which specific field are you looking for, just so I can start looking.
@MBeat did you already manually update this contact? Just curious as I'm investigating.
Oct 3 2019
I couldn't reproduce locally, but based on what I saw stepping through the debugger, it looks like when there is no campaign or iterations, the campaign variable is undefined. There is logic for the campaign being null but not undefined. I assumed we'd handle both the same way and added an or campaign === undefined. @AndyRussG Let me know what you think!
Sep 25 2019
@AndyRussG It looks like you were working on this and put something out yesterday. Should it be in review/deployed?
Sep 24 2019
I went with MaxCampaignFallback because I thought it had a better ring to it than using Iterations. I also noticed there's another config variable named FallbackHideCookieDuration which seems unrelated to Campaign Fallback so this name might be more clear than MaxFallbackIterations. Another options is MaxFallbackCampaign but I might be overthinking it :).
Sep 23 2019
Hmm, we think this could be pulling the gift source and using and if conditional in the thank you letter.
Sep 19 2019
@Ejegg are you working on this and should it be moved to "Doing"?
@Eileen I updated this documentation based on my knowledge, but I wasn't sure where is_opt_out was on the Contact record..do we use it?
@Dstine I added those to the documentation a while back but didn't update the task.
Sep 11 2019
Should this be closed @AndyRussG? 3/4 of these are merged.
Sep 5 2019
I think this would have been deployed by now?
Sep 4 2019
Okay, I took a stab at this. @DStrine Want to take a look?
Sep 3 2019
@Eileenmcnaughton in starting this I realized I didn't know whether we used the On Hold field in Civi or do not solicit for those who unsubscribe?
Aug 29 2019
@AndyRussG Where are we with this? I'm thinking it's in the wrong column.
The error display was resolved in https://phabricator.wikimedia.org/T149240, in that it now says "You do not have the correct permissions to perform the requested action or the action itself was invalid." as opposed to a full stack trace. However, I wonder if it's worth displaying the specific error message when a user inputs an invalid format.
Aug 27 2019
Note: FR wants to do a test of adyen versus globalcollect and waiting on this is a minor blocker. It looks like it's a 1. @Ejegg do you know why this got prioritized for Sprint S?
Aug 15 2019
Yup @Ejegg I got it. I'll put up a patch tomorrow morning.
@Ejegg Oh! I think this is caused by the type being "number" as I've learned it happens when any value with a decimal place is entered.
@CCogdill_WMF Can you confirm whether you see the error message when you directly click on the "Donate Monthly" button versus just tabbing away? That's what I found in my testing but want to confirm it. Thanks!
I'm currently thinking this might be something from Firefox, because I noticed the red highlighting doesn't happen in Chrome and the errorHighlight class doesn't seem to have actually been applied. I'm trying to test one more idea.
There are two places I thought the initial error highlighting might be coming from: validation_input and ext.donationinterface.validation.js, but when I enter console.log statements in them, they are not logged.
Okay, so I've found more interesting behavior. Basically the error highlighting appears if I tab away from the box, but when I click "Donate Monthly" the error text appears. I've been trying to find where in the code the highlight is applied but haven't found it yet.
Aug 14 2019
@XenoRyet Want to take a look at this or ask someone else to review?
So I had a weird experience testing this locally. I saw the error on my first test. Then I added some logging and tested again, and strangely the error text showed up! Then I tried to test without the logging (though it seemed weird if that fixed it) and there was an unrelated bug on ingenico's side that prevented me from testing further.
Note definition of done, with Phab tasks listed, but not created.
Or anyone else who remembers what our plan was? @Ejegg remembers something about mustache templates.
@jgleeson what did we plan to do here? Please write it up if you have time before leaving for Wikimania.
Aug 13 2019
@Eileenmcnaughton It looks like you started this?
Aug 2 2019
Hmm, so one question. Are we just looking for the fields in the custom set "Communication"? Also, I agree that so far there is no obvious place in our documentation to put this, but that means we need to create one.
@DStrine Do you just want the list on the task or somewhere else?
Jul 26 2019
This also happened in 2017 with cid 1092521. The pending recurring contributions report (where I found this) only has 5 rows though so it's possible this is a limited issue.
Jul 10 2019
@Ejegg Has this been deployed?
@XenoRyet I see patches merged. Should this be moved to "Pending Deployment"?
@AndyRussG How you would you like this reviewed? I wasn't sure if this is what you and @jgleeson talked about yesterday, but it's a long train of patches and I'm wondering if we should start by +2ing the original patch, or work from the top down.
Jul 5 2019
@Ejegg What is the best way to test this change?
Notes from tech-talk review:
Jul 3 2019
We talked about this in fortnightly today. @NNichols If you search for Communication:Employer Name Regex [a-zA-Z] that will return only those with values. (Not Empty works too but is slow.)
Jun 28 2019
This was emotionally difficult but I went ahead and removed all his roles and blocked his user. @Eileenmcnaughton any thoughts on whether we should delete his user or keep it for historical records (as in any changes he might have made in Civi)?
Jun 27 2019
May 30 2019
May 13 2019
I didn't know this existed! I need to add a link on the documentation homepage.
May 8 2019
@chasemp Sorry this dropped of my radar! Finally done today. Thanks for the pings :).
May 7 2019
Apr 24 2019
Weird @Cstone looks like you correctly linked the task but it didn't show up here. Thanks!
@XenoRyet did I remember you talking about this yesterday in standup or was it another part of contribution_tracking?
@Cstone where is your patch for this that we were looking at?
Apr 19 2019
@Ejegg some of the errors were from Monday though.
@MBeat is #577898 a uk or swedish donor?