Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#15584 closed enhancement (fixed)

Support map for packageMap

Reported by: dylan Owned by: Rawld Gill
Priority: high Milestone: 1.8
Component: Loader Version: 1.8.0b1
Keywords: map, packageMap, AMD Cc:
Blocked By: Blocking:

Description

With AMD adding support for map to the spec (instead of the name packageMap), should we add support for both names to 1.8 to improve AMD interop?

Change History (14)

comment:1 Changed 7 years ago by Rawld Gill

Milestone: tbd → 1.8
Priority: undecided → blocker
Status: new → assigned

comment:2 Changed 7 years ago by bill

Milestone: 1.8 → future
Priority: blocker → high

Rawld said he wants to punt this to future, and that the packageMap isn't part of the spec yet (although I didn't know there even was an official spec).

comment:3 Changed 7 years ago by dylan

Milestone: future → 1.8
Priority: high → blocker

comment:4 Changed 7 years ago by dylan

Milestone: 1.8 → future
Priority: blocker → high

comment:5 Changed 7 years ago by bill

Milestone: future → 1.8

Rawld changed his mind.

comment:6 Changed 7 years ago by Rawld Gill

In [29244]:

improved dojo/date/locale to no hard-code the package name dojo; fixes #15678; refs #15584

comment:7 Changed 7 years ago by Rawld Gill

In [29245]:

improve dijit/_CssStateMixin to not assume only one registry; refs #15679; refs #15584

comment:8 Changed 7 years ago by Rawld Gill

In [29247]:

improved dojo/i18n to not hard-code the package name dojo; fixes #15680; refs #15584; !strict

comment:9 Changed 7 years ago by Rawld Gill

Resolution: → fixed
Status: assigned → closed

In [29249]:

implemented AMD map config; added and updated tests; packageMap remains for for 1.x; fixes #15584; !strict

comment:10 Changed 7 years ago by Rawld Gill

In [29250]:

cleaned up some tests; refs #15584

comment:11 Changed 7 years ago by Rawld Gill

In [29257]:

removed unused loader cacheId machinery; improve cache url lookup machinery to handle mapped modules; eliminate potential clash between module-ids and urls in loader cache; refs #15584; !strict

comment:12 Changed 7 years ago by Rawld Gill

Resolution: fixed
Status: closed → reopened

Upon further clarification today, AMD map is specified differently than implemented in [29249].

comment:13 Changed 7 years ago by Rawld Gill

Resolution: → fixed
Status: reopened → closed

In [29262]:

modified AMD map implementation to align with specification clarification discussed on AMD-implement today; fixes #15584; !strict

comment:14 Changed 7 years ago by Rawld Gill

In [29263]:

updated test files that go with AMD map implementation; refs #15584

Note: See TracTickets for help on using tickets.