Skip to content

docs: add v8 deprecations - #30598

Closed
kapunahelewong wants to merge 1 commit into
angular:masterfrom
kapunahelewong:v8-deprecations
Closed

kapunahelewong wants to merge 1 commit into
angular:masterfrom
kapunahelewong:v8-deprecations

Conversation

@kapunahelewong

Copy link
Copy Markdown
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.io application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Adds v8 deprecated features to deprecation guide.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@mary-poppins

Copy link
Copy Markdown

You can preview d155006 at https://pr30598-d155006.ngbuilds.io/.

@kara kara left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A good start! Sorry the doc was so vague (definitely implementation-focused rather than user-focused). Minor edits requested below

Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
@kara kara added the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label May 22, 2019
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
@brandonroberts brandonroberts modified the milestones: Backlog, version 8 May 22, 2019
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
@mary-poppins

Copy link
Copy Markdown

You can preview 5e31b86 at https://pr30598-5e31b86.ngbuilds.io/.

Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
@mary-poppins

Copy link
Copy Markdown

You can preview bfd4bbd at https://pr30598-bfd4bbd.ngbuilds.io/.

@mary-poppins

Copy link
Copy Markdown

You can preview be2c7ed at https://pr30598-be2c7ed.ngbuilds.io/.

Comment thread aio/content/guide/deprecations.md Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for your input on this @trotyl 😄

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Totally agree! @trotyl is great! 🙌🏼

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you both~

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Also, technically /deep/ and >>> can be removed, which are already forbidden in preprocessor, the only thing matters is ::ng-deep.

Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
Comment thread aio/content/guide/deprecations.md Resolved
@mary-poppins

Copy link
Copy Markdown

You can preview 5c067ab at https://pr30598-5c067ab.ngbuilds.io/.

Comment thread aio/content/guide/deprecations.md Resolved
@mary-poppins

Copy link
Copy Markdown

You can preview f039fd4 at https://pr30598-f039fd4.ngbuilds.io/.

@Splaktar Splaktar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@mary-poppins

Copy link
Copy Markdown

You can preview 1a91f26 at https://pr30598-1a91f26.ngbuilds.io/.

@kara kara left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, but we should ask @IgorMinar if there's more context to add on wtf*

@brandonroberts brandonroberts added the target: patch This PR is targeted for the next patch release label May 23, 2019
@kara kara added action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels May 23, 2019
@kara kara added the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label May 23, 2019
@jasonaden jasonaden closed this in b5c8939 May 23, 2019
jasonaden pushed a commit that referenced this pull request May 23, 2019
@kapunahelewong
kapunahelewong deleted the v8-deprecations branch June 20, 2019 18:19
@angular-automatic-lock-bot

Copy link
Copy Markdown

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 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes effort1: hours merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note risk: low target: patch This PR is targeted for the next patch release type: bug/fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants