Skip to content

[PowerPoint] (Common APIs) Add a sample for Office.Slide - #2608

Open
Alex Jerabek (AlexJerabek) wants to merge 1 commit into
mainfrom
AlexJ-Slide
Open

[PowerPoint] (Common APIs) Add a sample for Office.Slide#2608
Alex Jerabek (AlexJerabek) wants to merge 1 commit into
mainfrom
AlexJ-Slide

Conversation

@AlexJerabek

Copy link
Copy Markdown
Contributor

This PR maps a sample to a related API for better coverage. It also fixes a typo I spotted.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 711a914:

✅ Validation status: passed

File Status Preview URL Details
docs/code-snippets/office-snippets.yaml ✅Succeeded

For more details, please refer to the build report.

@alison-mk Alison McKay (alison-mk) 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.

Approved with one suggestion.

let firstSlideId = 0;
function gotoSelectedSlide() {
//Get currently selected slide's id
// Get currently selected slide's ID

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.

Suggested change
// Get currently selected slide's ID
// Get currently selected slide's ID.

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.

2 participants