Skip to content

remove section in samsara-tutorial.md#1985

Open
chankarina wants to merge 1 commit into
MicrosoftDocs:mainfrom
chankarina:patch-2
Open

remove section in samsara-tutorial.md#1985
chankarina wants to merge 1 commit into
MicrosoftDocs:mainfrom
chankarina:patch-2

Conversation

@chankarina

Copy link
Copy Markdown
Contributor

One section needs to be removed as we do not support that way of onboarding with Samsara SSO anymore.

@prmerger-automator

Copy link
Copy Markdown
Contributor

@chankarina : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@chankarina

Copy link
Copy Markdown
Contributor Author

hello again! @nguhiu and @omondiatieno - have one more change to remove one outdated section, let me know if need to make any changes.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 8b4d6d6:

✅ Validation status: passed

File Status Preview URL Details
docs/identity/saas-apps/samsara-tutorial.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards

Copy link
Copy Markdown
Contributor

@nguhiu

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Removes the "Add Samsara from the gallery" section from the Samsara SSO tutorial because that onboarding path is no longer supported.

Changes:

  • Deletes the gallery-based app addition instructions and the related Enterprise App Configuration Wizard reference.

@chankarina

Copy link
Copy Markdown
Contributor Author

@nguhiu please let me know if theres anything else to change here - just removing a section from doc that is no longer relevant, thank you!

@chankarina

Copy link
Copy Markdown
Contributor Author

@nguhiu please let me know if theres anything else to change here - just removing a section from doc that is no longer relevant, thank you!

hi @nguhiu @omondiatieno - bumping this request - please lmk if there are additional changes that need to be made before this can be merged

@omondiatieno

Copy link
Copy Markdown
Contributor

@chankarina, thanks for your contribution. I see you've removed the section that shows how to add the app from the gallery. How is the app supposed to get into the gallery now if we remove that step? I'll ask @jeevansd to review this before we sign off.

@chankarina

Copy link
Copy Markdown
Contributor Author

@chankarina, thanks for your contribution. I see you've removed the section that shows how to add the app from the gallery. How is the app supposed to get into the gallery now if we remove that step? I'll ask @jeevansd to review this before we sign off.

thanks for directing @jeevansd to this! our main issue is we made SSO support changes and our SSO config doesn't work via app from gallery anymore.
until we figure out the right changes needed to support adding app from gallery, want to make sure the docs reflect the right steps.

thanks in advance!

@chankarina

Copy link
Copy Markdown
Contributor Author

@chankarina, thanks for your contribution. I see you've removed the section that shows how to add the app from the gallery. How is the app supposed to get into the gallery now if we remove that step? I'll ask @jeevansd to review this before we sign off.

thanks for directing @jeevansd to this! our main issue is we made SSO support changes and our SSO config doesn't work via app from gallery anymore. until we figure out the right changes needed to support adding app from gallery, want to make sure the docs reflect the right steps.

thanks in advance!

hi @jeevansd - please let me know if any additional changes are needed. thanks in advance!

@chankarina

Copy link
Copy Markdown
Contributor Author

@chankarina, thanks for your contribution. I see you've removed the section that shows how to add the app from the gallery. How is the app supposed to get into the gallery now if we remove that step? I'll ask @jeevansd to review this before we sign off.

thanks for directing @jeevansd to this! our main issue is we made SSO support changes and our SSO config doesn't work via app from gallery anymore. until we figure out the right changes needed to support adding app from gallery, want to make sure the docs reflect the right steps.
thanks in advance!

hi @jeevansd - please let me know if any additional changes are needed. thanks in advance!

hi @omondiatieno @nguhiu @jeevansd please let me know next steps! thanks in advance!

@jeevansd

Copy link
Copy Markdown
Contributor

@chankarina I am suggesting that you should submit the app update request following the process outlined here https://learn.microsoft.com/en-us/entra/identity/enterprise-apps/v2-howto-app-gallery-listing#update-or-remove-the-application-from-the-gallery and then we can get your gallery application updated. If you feel you are not sure what changes are needed, then just provide the test account for our team and we can find out what we need to update to unblock our customers here.

@chankarina

chankarina commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

@chankarina I am suggesting that you should submit the app update request following the process outlined here https://learn.microsoft.com/en-us/entra/identity/enterprise-apps/v2-howto-app-gallery-listing#update-or-remove-the-application-from-the-gallery and then we can get your gallery application updated. If you feel you are not sure what changes are needed, then just provide the test account for our team and we can find out what we need to update to unblock our customers here.

@jeevansd is it possible to get this documentation updated first while we figure out app update logistics internally? we will update this documentation back once we have this sorted.

@jeevansd

Copy link
Copy Markdown
Contributor

@chankarina I am suggesting that you should submit the app update request following the process outlined here https://learn.microsoft.com/en-us/entra/identity/enterprise-apps/v2-howto-app-gallery-listing#update-or-remove-the-application-from-the-gallery and then we can get your gallery application updated. If you feel you are not sure what changes are needed, then just provide the test account for our team and we can find out what we need to update to unblock our customers here.

@jeevansd is it possible to get this documentation updated first while we figure out app update logistics internally? we will update this documentation back once we have this sorted.

Nope we cannot deviate from our documentation template as we have over ~2000+ similar documentations. I can help expedite this as much as possible.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants