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

Date on prior distributions on partner view should be scheduled date, not date of entry. #4856

Open
1 task
cielf opened this issue Dec 15, 2024 · 2 comments · May be fixed by #4860
Open
1 task

Date on prior distributions on partner view should be scheduled date, not date of entry. #4856

cielf opened this issue Dec 15, 2024 · 2 comments · May be fixed by #4860
Assignees

Comments

@cielf
Copy link
Collaborator

cielf commented Dec 15, 2024

Summary

Date of prior distributions on the bank's view of a partner should be scheduled date, not date of entry.

Why?

That's how the banks would look for them.

Details

To see this,
1/ sign in as [email protected].
2/ find or make a distribution that has a different "Initial Allcoation" (ie. date of entry) and Date of Distribution. Note the partner.
3/ Then go to Partner Agencies, then All Parnters
4/ Click on that partner
5/ Scroll to the bottom

In the "Prior Distribuitons" section, you will see that your distribution has the date of entry showing, instead of the Date of Distribuiton.

What we want:

The Date column in "Prior Distributions" should show the Date of Distribution, not the date the distribution was entered.

Criteria for completion

  • [ ] behaviour as described above
  • tests to confirm that behaviour
@cielf cielf added Help Wanted Groomed + open to all! Difficulty—Beginner labels Dec 15, 2024
@McEileen
Copy link
Contributor

I would be up for working on this.

@cielf
Copy link
Collaborator Author

cielf commented Dec 16, 2024

Sounds good.

@github-actions github-actions bot removed the Help Wanted Groomed + open to all! label Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants