Skip to content

Main to live - 5/21/25 #3456

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 3 commits into from
May 21, 2025
Merged

Main to live - 5/21/25 #3456

merged 3 commits into from
May 21, 2025

Conversation

captainbrosset
Copy link
Contributor

@captainbrosset captainbrosset commented May 21, 2025

* draft

* update per tracked changes

* update detailed contents

* Updated app-actions.md with latest link and addressed to-dos

- all links are public links and removed all link related to-dos
- in the remaining to-do comments, please check the '05/19' for what's updated.

* trim LMC links

* leading spaces

* update Detailed contents

* Update app-actions.md to include sample app link

1. sample app Wami's link was missing and fixed. 
2. Fixed Web Manifest validation's in-doc link by adding the correct H6 heading name.

* linkfix

* Update microsoft-edge/progressive-web-apps-chromium/how-to/app-actions.md

change to lowercase formData

Co-authored-by: Michael Hoffman <michael.hoffman@collabera.com>

* incorp replies to PR Comments

* stray . in JSON

* present tense

* change -> to >

* Update microsoft-edge/progressive-web-apps-chromium/how-to/app-actions.md

commit suggested change of the action naming schema.

Co-authored-by: Patrick Brosset <patrickbrosset@gmail.com>

* Update microsoft-edge/progressive-web-apps-chromium/how-to/app-actions.md

Adding Windows doc link for more information at the intro

* Update microsoft-edge/progressive-web-apps-chromium/how-to/app-actions.md

Action changed to lowercase and plural

* Update microsoft-edge/progressive-web-apps-chromium/how-to/app-actions.md

Fixing the in-doc link name

* Update app-actions.md

Removing unnecessary notes. The instructions already said must so removing "[Required]".

* Update app-actions.md

Adding reference to PWA how-to doc

* Update microsoft-edge/progressive-web-apps-chromium/how-to/app-actions.md

Adding another reference to PWA howto doc

* Update microsoft-edge/progressive-web-apps-chromium/how-to/app-actions.md

Removing "[Required]" as it's redundant

* Update microsoft-edge/progressive-web-apps-chromium/how-to/app-actions.md

Removing "[Optional]"

* Update microsoft-edge/progressive-web-apps-chromium/how-to/app-actions.md

removing the heading 'Using Windows actions test tool', just show the numbered list.

* Update microsoft-edge/progressive-web-apps-chromium/how-to/app-actions.md

Revise the wording of the headings to avoid gerunds and question marks. Using verb and shortened the phrase.

* Update microsoft-edge/progressive-web-apps-chromium/how-to/app-actions.md

Revise the wording of the headings to avoid gerunds and question marks. Using verb and shortened the phrase.

* Update microsoft-edge/progressive-web-apps-chromium/how-to/app-actions.md

Revise the wording of the headings to avoid gerunds and question marks. Using verb and shortened the phrase.

* Update microsoft-edge/progressive-web-apps-chromium/how-to/app-actions.md

Revise the wording of the headings to avoid gerunds and question marks. Using verb and shortened the phrase.

* incorp own comments

* linkfix

* generic file name for actions manifest

* condense heading

* linkfix 2

* addressing the comments and adjust must-have H4 headings

* fix an inaccurate statement in prerequisites.

* Detailed contents

* incorp reviews

* App Action

* web app manifest

* Canary-bind

* ` > `

---------

Co-authored-by: Lingyan <6320831+novac42@users.noreply.github.com>
Co-authored-by: Patrick Brosset <patrickbrosset@gmail.com>
Copy link
Collaborator

@mikehoffms mikehoffms left a comment

Choose a reason for hiding this comment

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

LGTM - Writer/Editor

* landing, whether

* whether: 2 items

* fix Detailed Contents for Prompt

* intro hyphen

* sync toc title: Prompt

* bold list items

* global normize emdashes

* resize png

* optionally change

* demo page step sequence
* first draft

* add TOC

* indent links

* adorners

* removed chrome links

---------

Co-authored-by: Michael S. Hoffman <v-mihoffman@microsoft.com>
Copy link

@captainbrosset captainbrosset merged commit 714758b into live May 21, 2025
2 checks passed
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