WordPress / wordcamp.org

WordCamps are casual, locally-organized conferences covering everything related to WordPress.
https://wordcamp.org
132 stars 74 forks source link

Sponsor Invoices: Emails only go to Invoice requestor #708

Closed CdrMarks closed 1 year ago

CdrMarks commented 3 years ago

Describe the bug

Emailed noticed about the generation and payment of invoices are only sent to the user that created the invoice.

To reproduce

Steps to reproduce the behavior:

  1. As a WordCamp site user, create an invoice
  2. Have Harmony or someone with her capabilities approve or issue an invoice
  3. Receive an email at the email address associated with the invoice requestor

Expected behavior

I was expecting these notices to be sent to the WordCamp site's Administration Email Address

iandunn commented 3 years ago

That's the intended behavior, rather than a bug:

https://github.com/WordPress/wordcamp.org/blob/055fd1a66f036da5211ba489dc80a65e3d3a8081/public_html/wp-content/plugins/wordcamp-payments-network/includes/sponsor-invoices-dashboard.php#L357

It makes sense to me that the person who created the invoice would be the one who manages the rest of the process for it, but I'm guessing you use a different workflow?

Or is the goal to have the notices archived in a shared email box that is linked to the site admin address?

Maybe CC'ing the site admin address would solve this use case without disrupting other workflows where the author expects to get it directly?

CdrMarks commented 3 years ago

I opened this as a bug since I could only find these emailed notices in the email mailbox tied to my WPORG account, but I expected to see them in the shared WordCamp Google account we have.

Should I close this and reopen as a feature request?

iandunn commented 3 years ago

No, it's totally fine like this, I just mentioned that to help frame the discussion as adding support for this use case, rather than replacing the current behavior.

CdrMarks commented 2 years ago

I know this is still open and I've not had a chance to submit a PR for it yet.

Adding a note that I happened to see an email generated by send_invoice_pending_reminder_mail( $invoice_id, $organizer_mail ) which was also only sent to the user the created the sponsor. It's interesting, though, that this email's salutation was, "Howdy organizers".

timiwahalahti commented 1 year ago

Related #641

kcristiano commented 1 year ago

Just noting the issue was mentioned today - https://wordpress.slack.com/archives/C04L4HPFGHX/p1694624631633049

Email is not 100% so we'll always need copies. Organizing teams sometimes have Orgs that leave so linking to the creator's email is less than optimal. I do think an improvement would be to send them to the camps city@wordcamp.org email.

Next steps might include storing invoice number in the CPT that would let Organizers know it was generated and give those who can get copies a quicker way to find them.

If we can also open another ticket about allowing download by the organizers That would be very helpful. As these are transactional in nature, I don't really see any issues with WC admins being able to do that.

Before I look at the next steps I'd like to get this issue closed as wontfix or adopted.

timiwahalahti commented 1 year ago

Next steps might include storing invoice number in the CPT that would let Organizers know it was generated and give those who can get copies a quicker way to find them.

On the invoice post, there actually should already be a link to QBO for super deputies.

kcristiano commented 1 year ago

@timiwahalahti Is the link working? Wouldn't all super deputies need QBO access? @harmonyromo Can we (or did we) set up View only access in QBO?

harmonyromo commented 1 year ago

I think the only "viewing" user role would be just to see reports. There are other specific ones related to bill payment and expenses, but that wouldn't help here. I will do a bit more digging into this to be sure.

timiwahalahti commented 1 year ago

The link works for people with QBO access. Deputies being able to see invoice numbers might help sometimes, even though they cannot see the invoice or access QBO itself(?). Saving the invoice number and showing it would be the first step for #1060.