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.

Sign up for GitHub

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

Jump to bottom

Mail merge sends incorrect data for fields #2493

Closed
felipemillan opened this issue Jun 20, 2016 · 5 comments
Closed

Mail merge sends incorrect data for fields #2493

felipemillan opened this issue Jun 20, 2016 · 5 comments
Labels
bug needs-attention

Comments

@felipemillan
Copy link

The mass mail plugin it's an EPIC FAIL and I'm sad because I freaking love N1 but this gave me a MAJOR PROBLEM at work.

I have to send 55 emails saying SORRY but my email client made a mistake with the names and companies and details so SORRY.

This can't happen if you have a paid service.

Again I'm a big supporter of N1 but please take care of this issues as they are way to big for a paid service.

The main issue was that I uploaded a 55 email list with 5 columns
name, lastname, email, company and country and after I sent the email I realize that all the emails has the same name and then all of them have the same company so everyone got a "personalized email" with a different name and company but the last name was ok.
@bengotow
Copy link
Contributor

Hey! Can you clarify a bit what happened? Could you upload a screenshot of the composer with the tokens in there? Were there variables you dragged into the editor that were not converted into the values from the spreadsheet?

@jackiehluo jackiehluo changed the title Mass maling FAIL PLUGIN Mail merge sends incorrect data for fields Jun 30, 2016
@jackiehluo
Copy link
Contributor

Another incidence from Twitter DM:

Hi, friends! I'm a huge fan of Nylas. I recently had an issue though—I sent out a bulk email using Mail Merge and it ended up sending the same email to all 500 people with the wrong name and company name variables. I did the drag and drop, then added the CSV and double checked everything, and it looks right. Then, when I sent, it only used the variables of the first person for all email addresses. Is this a know issue?
Now that I've sent those messages (caps made it take a full 24 hours, even as I manually went through and tried to update each wrong variable before they send from drafts) and now Nylas is running incredibly. Messages aren't archiving when I do so from another app (Spark on iOS) and messages aren't sending until I restart Nylas.

@emorikawa
Copy link
Contributor

@grinich Has also reported this issue. Continuing to look into this.

@jackiehluo
Copy link
Contributor

Another report from @bengotow. 😞

jstejada added a commit that referenced this issue Jul 22, 2016
@jstejada
fix(mail-merge): Keep column links state properly in sync #2493
7b02076 
Summary:
See https://phab.nylas.com/D3112 for full description
- Update EditorAPI.insertCustomComponent to take a pre-generated
anchorId

Test Plan: Unit tests + manual

Reviewers: bengotow, halla, jackie

Reviewed By: jackie

Differential Revision: https://phab.nylas.com/D3113
@jackiehluo
Copy link
Contributor

Hey! We've pushed an update (0.4.47) that should fix this issue—let us know if you see it again! Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug needs-attention
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@emorikawa @bengotow @jstejada @felipemillan @jackiehluo