Feature #16944
Document how to get donation receipts
0%
Description
We’ve have 1 such request for Riseup Labs recently and it made me realize that we’re not explaining how to get them on /donate.
Subtasks
History
#1 Updated by sajolida 2019-08-07 12:08:29
- Parent task set to
Feature #16096
#2 Updated by sajolida 2019-08-26 18:18:16
- Status changed from Confirmed to Needs Validation
- Assignee changed from sajolida to moire
- Feature Branch set to web/16944-donation-receipts
Please have a look. We had 1 such request until now and we forwarded it to Riseup Labs, so let’s keep it simple.
#3 Updated by intrigeri 2019-09-04 14:04:12
- Target version changed from Tails_3.16 to Tails_3.17
#4 Updated by moire 2019-09-06 10:05:55
- Assignee changed from moire to sajolida
Just reviewed it.
Main issue:
- I could only test the “dollar-tax”. How can I trick the page to show me the “euro-tax” version?
Possible fix:
- classes name: tax-deducible -> tax-deductible
Remarks:
- The page is quite complex, so as a shortcut, I ctrl-f-ed for “receipt” and could not find the folded paragraph, then double-checked the code, but I think a normal reader will naturally find “tax-deductible”.
- Maybe the “.” after the email address is superfluous.
Otherwise it looks good!
#5 Updated by sajolida 2019-09-10 18:35:21
Thanks for the very careful review :)
> Main issue:
> - I could only test the “dollar-tax”. How can I trick the page to show me the “euro-tax” version?
I should have mentioned that to you. We coded the ‘euro-tax’ version
when thought that people would be able to donate in euros as well but
it’s dead code right now. We kept it around “just in case”, ha ha.
> Possible fix:
> - classes name: tax-deducible -> tax-deductible
Interestingly ‘deducible’ exists but means something totally different:
it comes from ‘deduce’ instead of ‘deduct’. I fixed a bunch more typos
in 07f39ab1c8.
> - The page is quite complex, so as a shortcut, I ctrl-f-ed for “receipt” and could not find the folded paragraph, then double-checked the code, but I think a normal reader will naturally find “tax-deductible”.
Right. We’ve had so little requests for receipts until now that I
thought that taking more space for everyone wasn’t worth it. These
instructions might become more complex in the future, so I thought that
the toggle section was a good investment, even if it’s only for 1
sentence right now. We might change our mind if many more people start
asking us for receipts now.
> - Maybe the “.” after the email address is superfluous.
I like it, it’s closing the sentence :)
#6 Updated by sajolida 2019-09-10 18:37:01
- Status changed from Needs Validation to Resolved
- Assignee deleted (
sajolida)
Merged!
#7 Updated by intrigeri 2019-09-12 14:27:47
- Target version changed from Tails_3.17 to Tails_4.0