Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exp date on view recurring contribution series -> 1969-12 #342

Closed
KarinG opened this issue Nov 26, 2020 · 3 comments
Closed

Exp date on view recurring contribution series -> 1969-12 #342

KarinG opened this issue Nov 26, 2020 · 3 comments

Comments

@KarinG
Copy link
Contributor

KarinG commented Nov 26, 2020

image

Expiry in this screen was only the card used to initiate the recurring series transaction -> hit View again (at the bottom there) to see the current card on file details.

But 1969-12 is definitively a bug that can be fixed - not at all critical though.

@adixon
Copy link
Contributor

adixon commented Nov 26, 2020

Noted. I'm not at all sure that field should still exist, core does some tracking of expiry dates and that card on file link will pull up the real expiry date.

FWIW - 1969-12 being generated from an empty unix timestamp.

@KarinG
Copy link
Contributor Author

KarinG commented Nov 26, 2020

I know and I agree -> best we don't store that field at all - after a few years it has no added value to know what the expiry date was the day that donor started their recurring contribution series. We can keep it on the list of things to do for a next round of updates.

@adixon
Copy link
Contributor

adixon commented Mar 18, 2021

Fixed with 6aef38a

Fixed by showing N/A if the expiry date string is empty.

In fact, we're not storing it, the civicrm payment token table is, and it does get updated when civicrm is used to update card credentials, so it's not a terrible thing. All I'm doing with this piece of code is exposing that field to the view of the recurring schedule, for credit cards, which at some point might be something core could do.

@adixon adixon closed this as completed Mar 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants