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

ampPileChangedRecipients sometimes doesn't work for replies #14

Open
tylermenezes opened this issue Feb 8, 2015 · 0 comments
Open

ampPileChangedRecipients sometimes doesn't work for replies #14

tylermenezes opened this issue Feb 8, 2015 · 0 comments

Comments

@tylermenezes
Copy link

When opening the composer to reply to a message, AMPComposerInfo.localMessage.GetMails only returns the "from" address. This leads to bugs like #8.

Repro

  1. [email protected] receives message from [email protected]
  2. Hit "reply" in Airmail

Expected: Airmail calls ampPileChangedRecipients with [info.localMessage GetMails] set to an array of one NSString: "[email protected]"

Actual: Airmail calls ampPileChangedRecipients with [info.localMessage GetMails] set to an array of one NSString: "[email protected]"


Subsequent calls work as expected, and checking after a delay works as expected. For example, this code will re-run ampPileChangedRecipients with the correct data after a short delay (which solves #8):

- (NSArray*) ampPileComposerView:(AMPComposerInfo *)info
{   
    dispatch_after(dispatch_time(DISPATCH_TIME_NOW, 10 * NSEC_PER_SEC), dispatch_get_main_queue(), ^{
        [self ampPileChangedRecipients:info];
    });

    return @[];
}
tylermenezes added a commit to tylermenezes/AirmailPlugIn-Framework that referenced this issue Feb 8, 2015
Fixes bug Airmail#8 with a hack around Airmail#14 using a timer.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant