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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(outlook): Fix Outlook integration #2047

Merged
merged 2 commits into from
Feb 16, 2022
Merged

Conversation

nunofmn
Copy link
Contributor

@nunofmn nunofmn commented Feb 7, 2022

馃専 What does this PR do?

  • 01d1539 - Fix Outlook integration

馃悰 Recommendations for testing

  • Check if the button shows up correctly in Outlook. 馃憣

馃摑 Links to relevant issues or information

Closes #2045

@nunofmn nunofmn requested a review from a team February 7, 2022 17:54
@nunofmn nunofmn self-assigned this Feb 7, 2022
Copy link
Contributor

@EternallLight EternallLight left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's back here, too! 馃檶馃徎

Just add the missing header to fix the build and 馃殌

src/content/outlook.js Show resolved Hide resolved
@hazem3500
Copy link

This PR should be g2g after changing the use strict header, right? 馃

@EternallLight EternallLight merged commit 9d14efb into master Feb 16, 2022
@EternallLight EternallLight deleted the 2045/outlook-fix branch February 16, 2022 14:25
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

Successfully merging this pull request may close these issues.

Outlook Integration Gone
3 participants