Lab E7: enable the new connection in the All vertical before testing search - #1018
Open
Elliot Margot (OwnOptic) wants to merge 1 commit into
Open
Elliot Margot (OwnOptic) wants to merge 1 commit into
Elliot Margot (OwnOptic) wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #753.
Exercise 1 Step 3 sends the learner to Microsoft 365 search to confirm the connector data is indexed, but a custom Copilot connector is not included in the All vertical until an admin enables it. The search comes back empty and the lab looks broken at exactly the point where it should look like it worked. #753 reported this in December after hitting it on a real tenant.
Added a note before the search step with the admin path and the two references it rests on:
I left out the search vertical step that #753 also suggests. Inline results in the All vertical need no custom vertical, and a vertical created without a result type shows no results at all, so it is a longer path than this step needs. The sample connector's schema already maps the
titlesemantic label, which is the other documented prerequisite for inline results.English source only, so the Japanese page is left for the translation workflow.
Verified with
mkdocs build: the site builds and the admonition renders on the E7 page with both links resolving.