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

Custom user guide link in enterprise welcome mail #2687

Conversation

HugsDaniel
Copy link
Contributor

What? Why?

Closes #2631

Following #2686
Adding dynamic user guide link to enterprise welcome email.

What should we test?

In the enterprise welcome email, the custom user guide link should be displayed.

Changelog Category: Changed

@HugsDaniel HugsDaniel force-pushed the 2631_user_guide_link_in_welcome_mail branch from 6f09ba9 to 4e67566 Compare September 12, 2018 11:03
@sauloperez
Copy link
Contributor

sauloperez commented Sep 12, 2018

I don't understand the difference with #2686. I see the exact same code.

@HugsDaniel
Copy link
Contributor Author

There's one more commit but I guess I could put it directly in #2686 since it's not a major modif

@sauloperez
Copy link
Contributor

You can replace the base of this PR to that other branch if that is what it is.

@mkllnk
Copy link
Member

mkllnk commented Sep 13, 2018

Now you continued working on #2686. Can we close this one?

@HugsDaniel
Copy link
Contributor Author

Yes ! I close it now

@HugsDaniel HugsDaniel closed this Sep 13, 2018
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