Problem/Motivation

ContextOverviewController is a stub. I imagined it as an important piece of UI. For example, for a user, it would form the basis of what was presented under a 'Gift Aid' tab. There is also DeclarationContextOverviewForm which seems related/similar.

We need to:
(a) figure out how it should it work, for donors and for staff
(b) figure out what's core and what's specific to the context so belongs in e.g. gift_aid_user
(c) fix it

Proposed resolution

In our updated model, this code relates to the donor entity. When viewing that entity we would like to see:

  1. current gift aid status summary
  2. buttons for actions (depends on context which are available): declare; cancel (will be done in #3505790: Declaration cancellation); edit => change address or add misc correspondence (staff); send written record (staff)
  3. list of all declarations (staff)

Add a gift aid tab for a user (sub-module) that is equivalent to viewing a Donor entity with context equal to the user..

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork gift_aid-3505330

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

jonathanshaw created an issue. See original summary.

adamps’s picture

Title: Fix GiftAidOverviewController » Additional information when viewing Donor entity
Issue summary: View changes
adamps’s picture

Issue summary: View changes
adamps’s picture

Issue summary: View changes

Closed #3505467: Finish the declaration forms as a duplicate of this. The only extra thing it had was the addition of the gift aid user tab, which I just added to the IS here.

adamps’s picture

Assigned: Unassigned » jonathanshaw
Issue summary: View changes
Status: Active » Needs review

OK to check this in? Then I'll update the roadmap, and I'm ready for you to have a look / test. After that I suggest we have another planning discussion.

adamps’s picture

I did the roadmap. Shall I create a first alpha release for you to test with?

jonathanshaw’s picture

Sure, pls do an alpha

  • adamps committed f74fbe36 on 1.x
    Issue #3505330 by adamps, jonathanshaw: Additional information when...
adamps’s picture

Status: Needs review » Fixed

Thanks. Done Alpha.

adamps’s picture

There is now a second alpha as the first was missing this issue. Sorry I forgot to "git pull" after switching back to 1.x to add the tag.

adamps’s picture

Assigned: jonathanshaw » Unassigned

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.