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

Make mailing name and campaign editable in place on Find Mailings #24698

Merged

Conversation

larssandergreen
Copy link
Contributor

Overview

There is currently no way to edit the mailing name or campaign for a scheduled, sent or archived mailing in the UI. If you have a naming system for mailings and someone sends a mailing that doesn't conform, you can't fix it and you can't change the campaign if it is incorrect.

Before

Mailing name and campaign not editable.

After

image

Mailing name and campaign editable in place on Find Mailings.

Technical Details

The id for the tr was changed from crm-mailing_{$row.id}, which does not appear to be used for anything, to mailing-{$row.id}.

@civibot
Copy link

civibot bot commented Oct 8, 2022

(Standard links)

@civibot civibot bot added the master label Oct 8, 2022
@larssandergreen larssandergreen changed the title Make mailing name and campaign editable in place in listings Make mailing name and campaign editable in place on Find Mailings Oct 8, 2022
@demeritcowboy
Copy link
Contributor

This looks good. Just wondering why was the crm-mailing id a problem? If somebody was targeting it for styling or something this would break that.

@colemanw
Copy link
Member

The in-place-edit jQuery code expects a dom id of entity name dash id. I think it's a safe enough change.

@colemanw colemanw merged commit 5b64ed3 into civicrm:master Oct 11, 2022
@larssandergreen larssandergreen deleted the editable-mailing-name-and-campaign branch November 5, 2022 00:09
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