Skip to content

Add extra dropdowns for every year in payment history#3840

Open
BPC2004 wants to merge 12 commits intomasterfrom
3839-view-all-payments
Open

Add extra dropdowns for every year in payment history#3840
BPC2004 wants to merge 12 commits intomasterfrom
3839-view-all-payments

Conversation

@BPC2004
Copy link
Contributor

@BPC2004 BPC2004 commented Nov 13, 2024

Closes #3839.

Summary

Made it so you can look through your payment history with the GUI.

How to test

  1. Go to account and then view payments.
  2. Then open all the dropdown menus and see if the links work.

@BPC2004 BPC2004 added the feature Issues regarding a complete new feature label Nov 13, 2024
@BPC2004 BPC2004 force-pushed the 3839-view-all-payments branch from 725e5fa to 0697211 Compare November 13, 2024 22:52
filters = []
for i in range(13):

for i in range(85):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

85 is kind of a weird number?... Is it based on the legal storage period for payment info? For the user it might be nice if years in which there were no payments (or at least years before their first payment) are not shown.

@BPC2004 BPC2004 force-pushed the 3839-view-all-payments branch 2 times, most recently from 4d6139b to 59af06a Compare April 9, 2025 18:23
Comment on lines 116 to +122
}
}
}

.wrapped-link {
word-wrap:break-word;
} No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not the right pr for these changes

Comment on lines 38 to 44
{% if partner.link %}
<h4>{% trans "Website" %}</h4>
<p>
<a href="{{ partner.link }}" rel="noopener" target="_blank">{{ partner.link }}</a>
<a href="{{ partner.link }}" rel="noopener" target="_blank" class="wrapped-link">{{ partner.link }}</a>
</p>
{% endif %}

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Issues regarding a complete new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable viewing entire payment history

4 participants