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

Added information about multitenant example #337

Merged
merged 2 commits into from Aug 22, 2018

Conversation

Projects
None yet
3 participants
@ardabeyazoglu
Contributor

ardabeyazoglu commented Aug 22, 2018

No description provided.

@o365devx

This comment has been minimized.

Show comment
Hide comment
@o365devx

o365devx Aug 22, 2018

Collaborator

Validation status: passed

File Status Preview URL Details
docs/develop/create-sso-office-add-ins-nodejs.md Succeeded

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Collaborator

o365devx commented Aug 22, 2018

Validation status: passed

File Status Preview URL Details
docs/develop/create-sso-office-add-ins-nodejs.md Succeeded

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@AlexJerabek

Thank you @ardabeyazoglu for putting together the multitenant sample. Great work!

My comments are just some small style/grammar things so all the Microsoft documentation sounds the same.

@o365devx

This comment has been minimized.

Show comment
Hide comment
@o365devx

o365devx Aug 22, 2018

Collaborator

Validation status: passed

File Status Preview URL Details
docs/develop/create-sso-office-add-ins-nodejs.md Succeeded

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Collaborator

o365devx commented Aug 22, 2018

Validation status: passed

File Status Preview URL Details
docs/develop/create-sso-office-add-ins-nodejs.md Succeeded

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@AlexJerabek AlexJerabek merged commit 91910cd into OfficeDev:master Aug 22, 2018

2 checks passed

OpenPublishing.Build Validation status: passed
Details
license/cla All CLA requirements met.
@AlexJerabek

This comment has been minimized.

Show comment
Hide comment
@AlexJerabek

AlexJerabek Aug 22, 2018

Contributor

Thank you again for this sample @ardabeyazoglu. Your changes to the docs will be merged into the live branch soon.

Contributor

AlexJerabek commented Aug 22, 2018

Thank you again for this sample @ardabeyazoglu. Your changes to the docs will be merged into the live branch soon.

@ardabeyazoglu

This comment has been minimized.

Show comment
Hide comment
@ardabeyazoglu

ardabeyazoglu Aug 22, 2018

Contributor

@AlexJerabek No problem. I've spent significant time to understand what is missing for multitenancy, at least others won't :)

Contributor

ardabeyazoglu commented Aug 22, 2018

@AlexJerabek No problem. I've spent significant time to understand what is missing for multitenancy, at least others won't :)

@ardabeyazoglu ardabeyazoglu deleted the ardabeyazoglu:patch-1 branch Aug 22, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment