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

CRM-19674 Add ability to see at a glance if a contribution is part of… #9421

Merged
merged 1 commit into from
Mar 17, 2017

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Nov 21, 2016

… a recurring series

recur

@pradpnayak
Copy link
Contributor

pradpnayak commented Dec 5, 2016

tested, working fine
screenshot from 2016-12-06 03 00 17

@eileenmcnaughton
Copy link
Contributor Author

@pradpnayak the customer has come back requesting the text (recurring payment) is cut down to just (R) - this is at @petednz suggestion I think - on the basis that it's less intrusive to eyeballing the numbers - what do you think? I'd need to make an edit to do that

… a recurring series

Note this is a subset of civicrm@c6ba3ef

It re-instates the display of the word recurring, but leaves out the extra column
for recurring & sortability based on that. This is mostly because I wanted to get the
PR resolved & merged & hence left out the stuff that did not
seem like I had a consensus on. However, I do think the loss
of sortability on recurring is a shame & hence kept a link to the
commit if anyone wants to revisit
@eileenmcnaughton
Copy link
Contributor Author

This got bogged down in discussion about the best approach & possible issues. I have reduced back to the minimum that was generally agreed to get it resolved & merged & closed. I will merge this on pass as this part was QA'd

Per my commit

Note this is a subset of c6ba3ef

It re-instates the display of the word recurring, but leaves out the extra column
for recurring & sortability based on that. This is mostly because I wanted to get the
PR resolved & merged & hence left out the stuff that did not
seem like I had a consensus on. However, I do think the loss
of sortability on recurring is a shame & hence kept a link to the
commit if anyone wants to revisit

@eileenmcnaughton eileenmcnaughton merged commit d372a58 into civicrm:master Mar 17, 2017
@eileenmcnaughton eileenmcnaughton deleted the recur_icon branch March 17, 2017 01:40
monishdeb pushed a commit to monishdeb/civicrm-core that referenced this pull request May 2, 2017
CRM-19674 Add ability to see at a glance if a contribution is part of…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants