Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add "Request headers (v2)" in "REST API Ref for Edge Add-ons" #3262

Merged
merged 8 commits into from
Sep 9, 2024
Merged

Add "Request headers (v2)" in "REST API Ref for Edge Add-ons" #3262

merged 8 commits into from
Sep 9, 2024

Conversation

yvsubhash
Copy link
Contributor

@yvsubhash yvsubhash commented Sep 5, 2024 •
edited by mikehoffms
Loading

Rendered article for review:

This PR adds v2 changes for the Edge Add-ons API.

AB#53687003

@mikehoffms mikehoffms changed the title Added request header changes for v2 version of the Api Added "Request headers v2" in "REST API Ref for Edge Add-ons" Sep 5, 2024
@mikehoffms mikehoffms changed the title Added "Request headers v2" in "REST API Ref for Edge Add-ons" Add "Request headers v2" in "REST API Ref for Edge Add-ons" Sep 5, 2024
mikehoffms
mikehoffms approved these changes Sep 5, 2024
View reviewed changes
Copy link
Contributor

@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.

Approved by Writer/Editor, after improved the v1 & v2 headings.

@mikehoffms mikehoffms changed the title Add "Request headers v2" in "REST API Ref for Edge Add-ons" Add "Request headers (v2)" in "REST API Ref for Edge Add-ons" Sep 5, 2024
@mikehoffms mikehoffms added the cat: extensions Extensions-related content. label Sep 5, 2024
dhruv16dhr
dhruv16dhr approved these changes Sep 6, 2024
View reviewed changes
Copy link

@dhruv16dhr dhruv16dhr left a comment

Choose a reason for hiding this comment

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

LGTM - Approved by Dev

yvsubhash
yvsubhash commented Sep 6, 2024
View reviewed changes
Copy link
Contributor Author

@yvsubhash yvsubhash left a comment

Choose a reason for hiding this comment

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

LGTM - approved

@learn-build-service-prod Learn Build Service (PROD)
Copy link

Learn Build status updates of commit 83c83fe:

✅ Validation status: passed

File Status Preview URL Details
microsoft-edge/extensions-chromium/publish/api/addons-api-reference.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@captainbrosset captainbrosset merged commit a0ef7ab into main Sep 9, 2024
2 checks passed
@captainbrosset captainbrosset deleted the user/syedugundla/docs/edgeaddonsapiv2 branch September 9, 2024 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mikehoffms mikehoffms mikehoffms approved these changes

@dhruv16dhr dhruv16dhr dhruv16dhr approved these changes

Assignees
No one assigned
Labels
cat: extensions Extensions-related content.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@yvsubhash @mikehoffms @dhruv16dhr @captainbrosset