Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Comments

chore: document deprecated webContents.getPrinters API#39356

Merged
codebytere merged 3 commits intomainfrom
properly-deprecate-get-printers
Aug 10, 2023
Merged

chore: document deprecated webContents.getPrinters API#39356
codebytere merged 3 commits intomainfrom
properly-deprecate-get-printers

Conversation

@dsanders11
Copy link
Member

@dsanders11 dsanders11 commented Aug 3, 2023

Description of Change

This API was marked in the docs as deprecated in #31023 (shipped in 17.0.0), but wasn't officially deprecated in docs/breaking-changes.md. Let's do a proper deprecation and remove it next version.

Checklist

Release Notes

Notes: The webContents.getPrinters API has been deprecated.

@dsanders11 dsanders11 added semver/minor backwards-compatible functionality target/26-x-y PR should also be added to the "26-x-y" branch. labels Aug 3, 2023
@dsanders11 dsanders11 requested a review from a team as a code owner August 3, 2023 14:51
Copy link
Member

@itsananderson itsananderson left a comment

Choose a reason for hiding this comment

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

API LGTM

@codebytere
Copy link
Member

API LGTM

Copy link
Contributor

@zcbenz zcbenz left a comment

Choose a reason for hiding this comment

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

API LGTM

Copy link
Member

@ckerr ckerr left a comment

Choose a reason for hiding this comment

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

API LGTM

@codebytere codebytere merged commit 11600c5 into main Aug 10, 2023
@codebytere codebytere deleted the properly-deprecate-get-printers branch August 10, 2023 09:50
@release-clerk
Copy link

release-clerk bot commented Aug 10, 2023

Release Notes Persisted

The webContents.getPrinters API has been deprecated.

@trop
Copy link
Contributor

trop bot commented Aug 10, 2023

I was unable to backport this PR to "26-x-y" cleanly;
you will need to perform this backport manually.

@trop trop bot added needs-manual-bp/26-x-y and removed target/26-x-y PR should also be added to the "26-x-y" branch. labels Aug 10, 2023
dsanders11 added a commit to dsanders11/electron that referenced this pull request Aug 10, 2023
* chore: document deprecated webContents.getPrinters API

* chore: remove duplicate deprecation warning
@trop
Copy link
Contributor

trop bot commented Aug 10, 2023

@dsanders11 has manually backported this PR to "26-x-y", please check out #39459

@trop trop bot added in-flight/26-x-y merged/26-x-y PR was merged to the "26-x-y" branch. and removed needs-manual-bp/26-x-y in-flight/26-x-y labels Aug 10, 2023
codebytere pushed a commit that referenced this pull request Aug 14, 2023
chore: document deprecated webContents.getPrinters API (#39356)

* chore: document deprecated webContents.getPrinters API

* chore: remove duplicate deprecation warning
win32ss pushed a commit to win32ss/supermium-electron that referenced this pull request Sep 24, 2023
* chore: document deprecated webContents.getPrinters API

* chore: remove duplicate deprecation warning
MrHuangJser pushed a commit to MrHuangJser/electron that referenced this pull request Dec 11, 2023
* chore: document deprecated webContents.getPrinters API

* chore: remove duplicate deprecation warning
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api-review/requested 🗳 merged/26-x-y PR was merged to the "26-x-y" branch. new-pr 🌱 PR opened recently semver/minor backwards-compatible functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants