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

AdminUI: add Manage Contribution Pages #24937

Merged
merged 1 commit into from
May 20, 2023

Conversation

aydun
Copy link
Contributor

@aydun aydun commented Nov 8, 2022

Overview

Convert 'Manage Contribution Pages' to SK

Before

Old page

After

New Page

Note:
- the temporary URL is civicrm/managecp

  • the original page has links to each tab of the edit screen. For consistency, the new page has a single 'Edit' button.

A few things to fix:

  1. In the help text, how should we add the help icon and link?
    2. The New Contribution Page does not work well as a pop-up. How can the 'Add Contribution Page' button be configured to open normally, not popup?
    3. The Contributions button should have 3 links, but including the token '[CURDATE()]' on the first two prevent them from loading.
    4. The Contribution searches for 'Current Month-To-Date' and 'Fiscal Year-To-Date' rely on this page to calculate the receive_date_low and receive_date_high. How can we calculate 'start of this month' and 'start of fiscal year' here? Or are those better done as presets on the contribution search page?
    5. The Clone button does not work.
    6. The original page has a 'Manage Personal Campaign Pages' button. Is there a better way than just editing the markup directly?
  2. It would look better with spacing between the 'Add Contribution Page' and 'Manage Personal Campaign Pages' buttons.

Technical Details

Schema xml changes to add paths for add, edit and delete and to specify the Contribution Page Title field as html text so it becomes available as filter.

Comments

@civibot
Copy link

civibot bot commented Nov 8, 2022

(Standard links)

@civibot civibot bot added the master label Nov 8, 2022
@colemanw
Copy link
Member

colemanw commented Nov 16, 2022

I tried this out and it basically works, which is great. Responses to your Qs:

  1. Help text icon, haven't been able to use those yet, but the documentation link should be easy enough to add, just put a normal link in the html.
  2. The "create new" and "manage personal campaign pages" are probably better off being done in the afform markup instead of as a property of the search display since it only does poups and it only does one button.
  3. Instead of 3 links to the old "Find contributions" screen how about 1 link to a new SearchKit screen?
  4. How about putting those search filters on a new SearchKit-based "Find contributions" screen and not trying to be too clever with the links.
  5. Yep, the clone button doesn't work. Seems like a problem with the formation of that link?
  6. Per #2 I'd say just edit the markup directly for those 2 buttons.

@civicrm-builder
Copy link

Can one of the admins verify this patch?

@aydun
Copy link
Contributor Author

aydun commented Apr 12, 2023

Hey @colemanw I've eventually got back around to this one!

1. Help text icon, haven't been able to use those yet, but the documentation link should be easy enough to add, just put a normal link in the html.

#24819 looks like it does some of the work to add in help links. Can we use that to link to existing text in .hlp files?

2. The "create new" and "manage personal campaign pages" are probably better off being done in the afform markup instead of as a property of the search display since it only does poups and it only does one button.

Done. Only thing is no spacing between the buttons. Is that just a css thing or am I missing something in the html?

3. Instead of 3 links to the old "Find contributions" screen how about 1 link to a new SearchKit screen?

For now, I've just linked to the existing 'Find contributions' screen with a param for the contribution_page_id

4. How about putting those search filters on a new SearchKit-based "Find contributions" screen and not trying to be too clever with the links.

Yes, can break that out as a separate thing.

5. Yep, the clone button doesn't work. Seems like a problem with the formation of that link?

Fixed

6. Per `#2` I'd say just edit the markup directly for those 2 buttons.

Done

@aydun aydun force-pushed the adminui_contribution_pages branch from 0ee8f01 to c2fffd4 Compare April 12, 2023 15:18
@aydun
Copy link
Contributor Author

aydun commented Apr 12, 2023

Looks like a dodgy test

Test this please

@aydun
Copy link
Contributor Author

aydun commented May 19, 2023

@colemanw I think there are just 2 things remaining here:

Item 1 above, about .hlp files and Item 2, spacing around buttons.

"contact_summary": null,
"summary_contact_type": null,
"icon": "fa-list-alt",
"server_route": "civicrm/managecp",
Copy link
Member

Choose a reason for hiding this comment

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

@aydun time to change the temp URL? I think this PR is mergeable. There are still issues but no blockers IMO.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed & squashed

@aydun aydun force-pushed the adminui_contribution_pages branch from c2fffd4 to 5e89766 Compare May 19, 2023 21:35
@aydun aydun changed the title WIP: AdminUI: add Manage Contribution Pages AdminUI: add Manage Contribution Pages May 19, 2023
@colemanw colemanw merged commit 8e9c8e6 into civicrm:master May 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants