Skip to content
This repository was archived by the owner on Dec 4, 2017. It is now read-only.

chore: remove moduleId from examples #3362

Merged
merged 6 commits into from
Mar 14, 2017

Conversation

Foxandxss
Copy link
Member

@Foxandxss Foxandxss commented Mar 8, 2017

I also removed some reactive forms live examples that got stuck in the repo, caught it sweeping moduleId.

  • Check plunkers
  • Check unit testing
  • AOT cookbook has a moduleId region. Probably need prose change
  • DELETE the cookbook about relative-paths (which is all about moduleId).
  • cookbook of ts to js could have prose changes as well.
  • Component styles guide has a test with moduleId in it.
  • Update style guide
  • Be sure that moduleId is not mentioned in any other cookbook.
  • Changelog entry

Copy link
Contributor

@wardbell wardbell left a comment

Choose a reason for hiding this comment

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

Really minor changes, then I'll approve and we merge.

Don't forget to delete the relative paths cookbook

@@ -21,7 +21,6 @@ export class SomeAbsoluteComponent {
// #docregion relative-config
@Component({
// #docregion module-id
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this #docregion still here?

Copy link
Member Author

Choose a reason for hiding this comment

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

as a reminder that something is talking about moduleid and needs my revision.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah ... and we are deleting this cookbook anyway.

@@ -9,7 +9,6 @@ import { HeroService } from './hero.service';
// #docregion renaming, metadata
@Component({
// #enddocregion renaming
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you can remove this

// #enddocregion renaming 

// #docregion renaming

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure, saw that, forgot to make a note.

Copy link
Contributor

@wardbell wardbell left a comment

Choose a reason for hiding this comment

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

Well done!

I only looked at the .jade files this round. I left you a few minor bits of text to fix before merging. No reason for me to hold back on approval but do fix them before merging.

@@ -5,6 +5,12 @@ block includes
The Angular documentation is a living document with continuous improvements.
This log calls attention to recent significant changes.

## Component relative paths cookbook has been deleted (2017-03-13)
The cookbook was well versed into moduleId, but there is no more moduleId in the guides.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we can combine lines 8 - 13 as

All mention of moduleId removed. "Component relative paths" cookbook deleted (2017-03-13)

We added a new SystemJS plugin (systemjs-angular-loader.js) to our recommended SystemJS configuration. This plugin dynamically converts "component-relative" paths in templateUrl and styleUrls to "absolute paths" for you.

We strongly encourage you to only write component-relative paths. That is the only form of URL discussed in these docs. You no longer need to write @Component({ moduleId: module.id }), nor should you.

@@ -255,8 +255,6 @@ block ts-decorator
Here are a few of the possible `@Component` configuration options:
Copy link
Contributor

Choose a reason for hiding this comment

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

Here are a few of the most useful @Component configuration options:

@@ -1679,10 +1679,6 @@ a(href="#toc") Back to top
:marked
**Do** name the style file `[component-name].component.css`, where [component-name] is the component name.

.s-rule.do
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we still need SOME of this:

.s-rule.do
:marked		
   **Do** specify _component-relative_ URLs, prefixed with `./`,

Just get rid of the moduleId stuff

@Foxandxss Foxandxss merged commit e6741ef into angular:master Mar 14, 2017
@Foxandxss Foxandxss deleted the bye-bye-module-id branch March 14, 2017 11:30
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.

3 participants