Skip to content

[excel] (Samples) Adjust how the samples and scenarios are branded #686

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

Merged
merged 6 commits into from
Dec 18, 2023

Conversation

AlexJerabek
Copy link
Collaborator

This PR adjusts the sample list and TOC to better delineate between types of samples. Most significantly, the Power Automate samples are separated so they're easier to find.

Copy link

Learn Build status updates of commit 019153c:

✅ Validation status: passed

File Status Preview URL Details
docs/index.md ✅Succeeded View
docs/resources/samples/add-excel-comments.md ✅Succeeded View
docs/resources/samples/range-samples.md ✅Succeeded View
docs/resources/samples/samples-overview.md ✅Succeeded View
docs/toc.yml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link

Learn Build status updates of commit 4aaa9aa:

✅ Validation status: passed

File Status Preview URL Details
docs/index.md ✅Succeeded View
docs/resources/samples/add-excel-comments.md ✅Succeeded View
docs/resources/samples/range-samples.md ✅Succeeded View
docs/resources/samples/samples-overview.md ✅Succeeded View
docs/toc.yml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

| [Cross-reference workbooks](excel-cross-reference.md) | This sample uses Office Scripts and Power Automate to cross-reference and validate information in different workbooks. |
| [Email chart and table images](email-images-chart-table.md) | This sample uses Office Scripts and Power Automate actions to create a chart and send that chart as an image by email. |
| [Record day-to-day changes in Excel and report them with a flow](report-day-to-day-changes.md) | This sample uses a scheduled Power Automate flow to record daily readings and report the changes. |
| [Run a script on all Excel files in a folder](automate-tasks-on-all-excel-files-in-folder.md) | This project performs a set of automation tasks on all files situated in a folder on OneDrive for Business (can also be used for a SharePoint folder). It performs calculations on the Excel files, adds formatting, and inserts a comment that @mentions a colleague. |
| [Write a large dataset](write-large-dataset.md) | This sample shows how to send a large range as smaller subranges. |
Copy link
Member

Choose a reason for hiding this comment

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

At first glance, I'm not sure it's clear what "send" means and how this scenario fits in the cross-application section?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fair point. I'll clarify the scenario.

"This sample shows how to work with a large range as smaller subranges. This helps flows handle data transfer limits and operation timeouts.

| [Cross-reference workbooks](excel-cross-reference.md) | This sample uses Office Scripts and Power Automate to cross-reference and validate information in different workbooks. |
| [Email chart and table images](email-images-chart-table.md) | This sample uses Office Scripts and Power Automate actions to create a chart and send that chart as an image by email. |
| [Record day-to-day changes in Excel and report them with a flow](report-day-to-day-changes.md) | This sample uses a scheduled Power Automate flow to record daily readings and report the changes. |
| [Run a script on all Excel files in a folder](automate-tasks-on-all-excel-files-in-folder.md) | This project performs a set of automation tasks on all files situated in a folder on OneDrive for Business (can also be used for a SharePoint folder). It performs calculations on the Excel files, adds formatting, and inserts a comment that @mentions a colleague. |
Copy link
Member

Choose a reason for hiding this comment

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

"This sample" instead of "this project" to be consistent?

Copy link
Collaborator

@ElizabethSamuel-MSFT ElizabethSamuel-MSFT left a comment

Choose a reason for hiding this comment

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

My suggestions are how I interpreted what the style guide says. Not sure about the list-y ones (the "dropdown lists, .." ones).

Co-authored-by: Elizabeth Samuel <elizs@microsoft.com>
Copy link

Learn Build status updates of commit 75f548d:

✅ Validation status: passed

File Status Preview URL Details
docs/index.md ✅Succeeded View
docs/resources/samples/add-excel-comments.md ✅Succeeded View
docs/resources/samples/range-samples.md ✅Succeeded View
docs/resources/samples/samples-overview.md ✅Succeeded View
docs/toc.yml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link

Learn Build status updates of commit 7a7f9e0:

✅ Validation status: passed

File Status Preview URL Details
docs/index.md ✅Succeeded View
docs/resources/samples/add-excel-comments.md ✅Succeeded View
docs/resources/samples/range-samples.md ✅Succeeded View
docs/resources/samples/samples-overview.md ✅Succeeded View
docs/toc.yml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@AlexJerabek AlexJerabek merged commit 997ebd6 into main Dec 18, 2023
@AlexJerabek AlexJerabek deleted the AlexJ-ScenarioTweaks branch December 18, 2023 19:00
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