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

Remove Email CTA Part 2 of 2 #2913

Merged
merged 6 commits into from
Mar 29, 2019
Merged

Remove Email CTA Part 2 of 2 #2913

merged 6 commits into from
Mar 29, 2019

Conversation

youriwims
Copy link
Contributor

@youriwims youriwims commented Mar 28, 2019

Closes #2899
Part 1 of 2

This pr removes the use of the email cta in theOpportunity template's model. Removing the snippet from the template was completed previously in Part 1 ↖️ .

To Test
Create new opportunity page in app and there will be no option to add email cta.

Checklist

Changes in Models:

  • Did I update or add new fake data?
  • If my changes are not backward-compatible, did I schedule a deploy with the rest of the team?

@cadecairos cadecairos temporarily deployed to foundation-mofostaging-pr-2913 March 28, 2019 15:05 Inactive
@youriwims youriwims requested a review from alanmoo March 28, 2019 15:07
@cadecairos cadecairos temporarily deployed to foundation-mofostaging-pr-2913 March 28, 2019 18:24 Inactive
@cadecairos cadecairos temporarily deployed to foundation-mofostaging-pr-2913 March 28, 2019 21:35 Inactive
@cadecairos cadecairos temporarily deployed to foundation-mofostaging-pr-2913 March 29, 2019 16:22 Inactive
@cadecairos cadecairos temporarily deployed to foundation-mofostaging-pr-2913 March 29, 2019 17:34 Inactive
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.

3 participants