-
Notifications
You must be signed in to change notification settings - Fork 27.4k
Conversation
Thanks @lgalfaso - these and the checkAnchors processor you wrote are going to improve the docs a lot. A number of these fixes are not optimal though. For instance most API entities, such as a service can be referenced in a |
let me look into this later today |
Also I have just refactored parts of and merged your angular/dgeni-packages#78 |
Fix many dangling links
There are a few pending things outside of this PR
Once dgeni is updated here, there is a need to update the configuration |
abdaab7
to
30996f8
Compare
@lgalfaso - are these pending things basically just docs things that need fixing or issues with this PR? |
@petebacondarwin all the pending things are out of the scope of this PR |
Shall we create new issues for them? (Or did you do that already?) |
I just saw that you already fixed some of the issues in your refactor at dgeni-packages. The remaining things are
The later point, will open a defect to remove it or for someone to write it |
Closing this based on the discussion above. |
No! The discussion is about things outside of the scope of this PR. This PR still needs to be merged |
Fix many dangling links