Skip to content

refactor(cdk/clipboard): clean up deprecated APIs for v13 #23401

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
Merged
merged 1 commit into from
Aug 20, 2021
Merged

refactor(cdk/clipboard): clean up deprecated APIs for v13 #23401

merged 1 commit into from
Aug 20, 2021

Conversation

@crisbeto
Copy link
Member

@crisbeto crisbeto commented Aug 18, 2021 •
edited
Loading

Cleans up the APIs that were marked for removal for v13.

BREAKING CHANGE:

  • CKD_COPY_TO_CLIPBOARD_CONFIG has been removed. Use CDK_COPY_TO_CLIPBOARD_CONFIG instead.

@crisbeto crisbeto added P2 The issue is important to a large percentage of users, with a workaround target: major This PR is targeted for the next major release labels Aug 18, 2021
@crisbeto crisbeto added this to the 13.0.0 milestone Aug 18, 2021
@crisbeto crisbeto requested a review from devversion August 18, 2021 19:20
@crisbeto crisbeto requested review from a team and jelbourn as code owners August 18, 2021 19:20
@google-cla google-cla bot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Aug 18, 2021
@itea-dev
Copy link

itea-dev commented Aug 18, 2021 •
edited
Loading

@crisbeto

There is a typo in the description.

Use CKD_COPY_TO_CLIPBOARD_CONFIG instead. 👉 Use CDK_COPY_TO_CLIPBOARD_CONFIG instead.

@crisbeto
refactor(cdk/clipboard): clean up deprecated APIs for v13
32a0e04 
Cleans up the APIs that were marked for removal for v13.

BREAKING CHANGE:
* `CKD_COPY_TO_CLIPBOARD_CONFIG` has been removed. Use `CDK_COPY_TO_CLIPBOARD_CONFIG` instead.
@crisbeto crisbeto force-pushed the v13-clipboard-cleanup branch from cf6aeda to 32a0e04 Compare August 18, 2021 20:23
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Aug 19, 2021
@amysorto amysorto merged commit 5833597 into angular:master Aug 20, 2021
@angular-automatic-lock-bot
Copy link

angular-automatic-lock-bot bot commented Sep 20, 2021

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@devversion devversion devversion approved these changes

@jelbourn jelbourn Awaiting requested review from jelbourn

Assignees

No one assigned

Labels

action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement P2 The issue is important to a large percentage of users, with a workaround target: major This PR is targeted for the next major release

Projects

None yet

Milestone

13.0.0

Development

Successfully merging this pull request may close these issues.

4 participants

@crisbeto @itea-dev @devversion @amysorto