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

Add customer's email address to "Order Line Items" report-format output #26

Closed
lat9 opened this issue Mar 14, 2021 · 1 comment
Closed

Comments

@lat9
Copy link
Owner

lat9 commented Mar 14, 2021

... as requested in this Zen Cart support-thread posting.

@lat9 lat9 added the change label Mar 14, 2021
@jage9
Copy link

jage9 commented Aug 24, 2021

I have made this mod on my store and can submit as a PR if desired.

  • Adds Email address column to table display and CSV for the order level reporting
  • Email is pulled from the order itself, so is the Email address as of the time of the order, consistent with name handling
  • Added appropriate language file entry for heading

If this messes with the visuals, could explore just making this available for the CSV, or an optional field, though there is not currently a way to hide plenty of other fields.

I looked at adding Email into the list of sort fields, but that feature seems to not be working as expected in 3.4.0 unless I'm misunderstanding it.

The major use case for me is to easily gather a list of Email addresses for an ordered product, to provide an update, recall, or other info. Perhaps it makes more sense to use the current customer Email address if available, but again the first/last from the order record is what's currently being used, so tried to remain consistent. One could argue this is a bit outside of the purview of a sales reporting mod, but it allows for easy customer follow-up based on sales, so I think it fits, at least until there is another solution. Certainly open to feedback.

@lat9 lat9 added this to the v4.0.0 milestone Apr 28, 2023
lat9 added a commit that referenced this issue Apr 28, 2023
... to cut down on memory required if the option's not needed.
@lat9 lat9 closed this as completed Apr 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants