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

Donation Site Screen improvement #4979

Open
3 tasks
cielf opened this issue Feb 2, 2025 · 2 comments · May be fixed by #4991
Open
3 tasks

Donation Site Screen improvement #4979

cielf opened this issue Feb 2, 2025 · 2 comments · May be fixed by #4991
Assignees

Comments

@cielf
Copy link
Collaborator

cielf commented Feb 2, 2025

Summary

Make donation site index page consistent with the rest of the system

Why?

It's a bit weird and out of place as it is.

Details

To reach the page:

sign in as [email protected]
click Community
Click Donation Sites

What we want done:

At the top of the page, there is a row of fields and a create button. Remove them. They are redundant, as the 'New Donation Site" takes you to a form for doing the same thing.
Why remove the row of fields instad of the new donation site button? Consistency with the rest of the system.

Criteria for completion

  • behaviour as described above
  • adjust tests to match behaviour (i.e. f there are any tests about the row of fields and create, remove them)
  • adjust user guide to match behaviour (docs/user_guide/bank/community_donation_sites.md)
@cielf cielf added Help Wanted Groomed + open to all! Difficulty—Beginner labels Feb 2, 2025
@Oce-ane
Copy link

Oce-ane commented Feb 2, 2025

Could I take this ?

@cielf
Copy link
Collaborator Author

cielf commented Feb 3, 2025

Yes, please.

@github-actions github-actions bot removed the Help Wanted Groomed + open to all! label Feb 3, 2025
@Oce-ane Oce-ane linked a pull request Feb 3, 2025 that will close this issue
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