-
Notifications
You must be signed in to change notification settings - Fork 3
Add country details for Denmark Djibouti Dominica #55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add country details for Denmark Djibouti Dominica #55
Conversation
| private const string DENMARK_ISO2_CODE = "DK"; | ||
| private const string DENMARK_ISO3_CODE = "DNK"; | ||
| private readonly string[] DENMARK_CALLING_CODE = ["+045"]; | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No state type test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
resolved
| Assert.Equal(DENMARK_NUMERIC_CODE, country.NumericCode); | ||
| Assert.Equal(DENMARK_ISO2_CODE, country.ISO2Code); | ||
| Assert.Equal(DENMARK_ISO3_CODE, country.ISO3Code); | ||
| Assert.Equal(DENMARK_CALLING_CODE, country.CallingCode); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No state type test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
resolved
| private const string DJIBOUTI_ISO2_CODE = "DJ"; | ||
| private const string DJIBOUTI_ISO3_CODE = "DJI"; | ||
| private readonly string[] DJIBOUTI_CALLING_CODE = ["+253"]; | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No state type test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
resolved
src/World.Net/Countries/Denmark.cs
Outdated
| public string ISO3Code { get; } = "DNK"; | ||
|
|
||
| ///<inheritdoc/> | ||
| public string[] CallingCode { get; } = ["+045"]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+45 without the 0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
resolved
| new("North Denmark", "DK-81", "Region"), | ||
| new("Southern Denmark", "DK-83", "Region"), | ||
| new("Zealand", "DK-85", "Region"), | ||
| ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
public IEnumerable States { get; } =
[
new("Central Denmark", "DK-82", "Region"),
new("Capital Region of Denmark", "DK-84", "Region"),
new("North Denmark", "DK-81", "Region"),
new("Southern Denmark", "DK-83", "Region"),
new("Zealand", "DK-85", "Region"),
];
Denmark isn't part of the list
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
resolved
src/World.Net/Countries/Dominica.cs
Outdated
| public string ISO3Code { get; } = "DMA"; | ||
|
|
||
| ///<inheritdoc/> | ||
| public string[] CallingCode { get; } = ["+1"]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use this +1-767
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
resolved
No description provided.