Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Docs known issues #16812

Merged
merged 5 commits into from
Jan 21, 2019
Merged

Docs known issues #16812

merged 5 commits into from
Jan 21, 2019

Conversation

Narretz
Copy link
Contributor

@Narretz Narretz commented Jan 14, 2019

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Updates various APIs with known issues that we are not planning to fix due to LTS / never intended to fix.

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change?

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Fix/Feature: Docs have been added/updated
  • Fix/Feature: Tests have been added; existing tests pass

Other information:

Copy link
Contributor

@petebacondarwin petebacondarwin left a comment

Choose a reason for hiding this comment

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

One typo

@jbedard
Copy link
Collaborator

jbedard commented Jan 14, 2019

Are there github issues for these? Would it be worth linking to them?

@Narretz
Copy link
Contributor Author

Narretz commented Jan 15, 2019

@jbedard You mean link them in the docs directly? The commits close the issues which are related.

@jbedard
Copy link
Collaborator

jbedard commented Jan 15, 2019

Oh I didn't notice all the issues being closed. That would be a lot of links to put into comments :/ Would those github pages ever have useful info though? Describing alternatives/workarounds etc...

@Narretz
Copy link
Contributor Author

Narretz commented Jan 15, 2019

Some have useful info, others don't. It's hard to make that call

@Narretz Narretz force-pushed the docs-known-issues branch 2 times, most recently from c67b1bc to f89a411 Compare January 16, 2019 21:16
@Narretz Narretz merged commit 65f800e into angular:master Jan 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants