| Nov | DEC | Jan |
| 08 | ||
| 2019 | 2020 | 2021 |
COLLECTED BY
Collection: github.com
|
|
Normally I am cautious about adding new functionality but I think that this case could be an exception. Since we are already doing the calculations for the week numbering, it is probably not too much more to compute the "week year".
This might be a good first time PR for someone... |
|
Hi, I'd like to work on this issue. |
|
@prashant-tholia - great! Please make sure you have read https://github.com/angular/angular/blob/master/CONTRIBUTING.md. Let me know if you need help getting started. |
|
Thanks @petebacondarwin! I'll make sure to fulfill all mentioned guidelines and requirements. |
|
Hello, can I also be work on this issue with @prashant-tholia since this is a good first issue and I am an open source beginner. |
feat(common): week-numbering year format support
8f012cc
Add support to format iso week-numbering year with 'r' in dateFormat string template. PR angular#38739
feat(common): week-numbering year format support
1e2b94d
Add support to format iso week-numbering year with 'r' in dateFormat string template. PR angular#38739prashanttholia added a commit to prashanttholia/angular that referenced this issue
feat(common): Add ISO week-numbering year formats support to formatDate
4d5a018
Add ISO 8601 week-numbering year formats ('r', 'rr', 'rrr', 'rrrr') support for formatDate function.
Issue:angular#38739
feat(common): Add ISO week-numbering year formats support to formatDate
6c8af9c
Add ISO 8601 week-numbering year formats ('r', 'rr', 'rrr', 'rrrr') support for formatDate function.
Issue:angular#38739
prashanttholia
added a commit
to prashanttholia/angular
that referenced
this issue
feat(common): Add ISO week-numbering year formats support to formatDate
c60b015
Add ISO 8601 week-numbering year formats ('r', 'rr', 'rrr', 'rrrr') support for formatDate function.
Issue:angular#38739
prashanttholia
added a commit
to prashanttholia/angular
that referenced
this issue
feat(common): Add ISO week-numbering year formats support to formatDate
ba74534
Add ISO 8601 week-numbering year formats ('r', 'rr', 'rrr', 'rrrr') support for formatDate function.
Issue:angular#38739
prashanttholia
added a commit
to prashanttholia/angular
that referenced
this issue
feat(common): Add ISO week-numbering year formats support to formatDate
a336319
Add ISO 8601 week-numbering year formats ('r', 'rr', 'rrr', 'rrrr') support for formatDate function.
Issue:angular#38739
prashanttholia
added a commit
to prashanttholia/angular
that referenced
this issue
feat(common): Add ISO week-numbering year formats support to formatDate
2a781cf
Add ISO 8601 week-numbering year formats ('r', 'rr', 'rrr', 'rrrr') support for formatDate function.
Issue:angular#38739
prashanttholia
added a commit
to prashanttholia/angular
that referenced
this issue
feat(common): Add ISO week-numbering year formats support to formatDate
a33de4c
Add ISO 8601 week-numbering year formats ('r', 'rr', 'rrr', 'rrrr') support for formatDate function.
Issue:angular#38739
prashanttholia
added a commit
to prashanttholia/angular
that referenced
this issue
feat(common): Add ISO week-numbering year formats support to formatDate
a891f57
Add ISO 8601 week-numbering year formats ('r', 'rr', 'rrr', 'rrrr') support for formatDate function.
Issue:angular#38739
mhevery
added a commit
that referenced
this issue
feat(common): Add ISO week-numbering year formats support to formatDa…
Loading status checks…
984ed39
…te (#38828)
Add ISO 8601 week-numbering year formats ('r', 'rr', 'rrr', 'rrrr') support for formatDate function.
Issue:#38739PRClose #38828
|
@prashanttholia, Not to be pedantic or anything, but I think the 'r' placeholder is a mistake here. CLDR/Unicode defines this in http://www.unicode.org/reports/tr35/tr35-dates.html#Date_Field_Symbol_Tableas
The ISO week year is actually 'Y', 'YY', 'YYY', 'YYYY', etc.:
|
|
@rubenprins, I apologize for the late response. I think that formats should be consistent with or as close as possible to the standard. I agree that 'Y' would be a more appropriate choice to represent week numbering year. @petebacondarwin, please let us know what you think and whether it needs to be updated? |
|
@prashanttholia - sorry for not replying sooner. We discussed this and felt it would be better to go with the (We need to work out what to do about ISO alignment for weeks etc too.) (Also thanks to @cexbrayat for pinging me about this outstanding question!) |
|
@petebacondarwin, no worries. I am really glad that it got fixed. I have mentioned my thoughts on the ISO alignment issue in the comment here. Please have a look. |
|
Thanks @prashanttholia |
Successfully merging a pull request may close this issue.
None yet 6 participants ●© 2020 GitHub, Inc. ●Terms ●Privacy ● ●Security ●Status ●Help ●Contact GitHub ●Pricing ●API ●Training ●Blog ●About You can’t perform that action at this time. You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.