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.

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

chore: document deprecated webContents.getPrinters API #39356

Merged
merged 3 commits into from Aug 10, 2023

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
@electron-cation electron-cation bot added api-review/requested 🗳 new-pr 🌱 PR opened in the last 24 hours labels Aug 3, 2023
Copy link
Contributor

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

@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
14 of 15 checks passed
@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 in the last 24 hours semver/minor backwards-compatible functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants