Skip to content

Commit

Permalink
Update API based on review feedback (#47435)
Browse files Browse the repository at this point in the history
  • Loading branch information
dubiety authored Dec 6, 2024
1 parent 5b27811 commit 529e8fa
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 16 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -77,9 +77,6 @@ public enum ServiceVersion
V1_0 = 1,
}
}
public static partial class MapsTimeZoneModelFactory
{
}
public static partial class MapsTimeZonesModelFactory
{
public static Azure.Maps.TimeZones.CountryRecord CountryRecord(string name = null, string code = null) { throw null; }
Expand All @@ -91,7 +88,7 @@ public static partial class MapsTimeZonesModelFactory
public partial class ReferenceTime
{
internal ReferenceTime() { }
public System.TimeSpan DaylightSavingsTimeSpan { get { throw null; } }
public System.TimeSpan DaylightSavingsOffset { get { throw null; } }
public string PosixTimeZone { get { throw null; } }
public int? PosixTimeZoneValidYear { get { throw null; } }
public System.TimeSpan StandardOffsetTimeSpan { get { throw null; } }
Expand All @@ -103,7 +100,7 @@ internal ReferenceTime() { }
public partial class TimeTransition
{
internal TimeTransition() { }
public System.TimeSpan DaylightSavingsTimeSpan { get { throw null; } }
public System.TimeSpan DaylightSavingsOffset { get { throw null; } }
public System.TimeSpan StandardOffsetTimeSpan { get { throw null; } }
public string Tag { get { throw null; } }
public System.DateTimeOffset? UtcEnd { get { throw null; } }
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -77,9 +77,6 @@ public enum ServiceVersion
V1_0 = 1,
}
}
public static partial class MapsTimeZoneModelFactory
{
}
public static partial class MapsTimeZonesModelFactory
{
public static Azure.Maps.TimeZones.CountryRecord CountryRecord(string name = null, string code = null) { throw null; }
Expand All @@ -91,7 +88,7 @@ public static partial class MapsTimeZonesModelFactory
public partial class ReferenceTime
{
internal ReferenceTime() { }
public System.TimeSpan DaylightSavingsTimeSpan { get { throw null; } }
public System.TimeSpan DaylightSavingsOffset { get { throw null; } }
public string PosixTimeZone { get { throw null; } }
public int? PosixTimeZoneValidYear { get { throw null; } }
public System.TimeSpan StandardOffsetTimeSpan { get { throw null; } }
Expand All @@ -103,7 +100,7 @@ internal ReferenceTime() { }
public partial class TimeTransition
{
internal TimeTransition() { }
public System.TimeSpan DaylightSavingsTimeSpan { get { throw null; } }
public System.TimeSpan DaylightSavingsOffset { get { throw null; } }
public System.TimeSpan StandardOffsetTimeSpan { get { throw null; } }
public string Tag { get { throw null; } }
public System.DateTimeOffset? UtcEnd { get { throw null; } }
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
namespace Azure.Maps.TimeZones
{
/// <summary> Model factory for models. </summary>
[CodeGenModel("MapsTimeZoneModelFactory")]
public static partial class MapsTimeZoneModelFactory
internal static partial class MapsTimeZoneModelFactory
{
}
}
4 changes: 2 additions & 2 deletions sdk/maps/Azure.Maps.TimeZones/src/Models/ReferenceTime.cs
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ internal ReferenceTime(string tag, string standardOffset, string daylightSavings
_daylightSavings = daylightSavings;
if (TimeSpan.TryParse(daylightSavings, out TimeSpan DaylightSavingsValue))
{
DaylightSavingsTimeSpan = DaylightSavingsValue;
DaylightSavingsOffset = DaylightSavingsValue;
}

_wallTime = wallTime;
Expand All @@ -55,7 +55,7 @@ internal ReferenceTime(string tag, string standardOffset, string daylightSavings
[CodeGenMember("DaylightSavings")]
internal string _daylightSavings { get; }
/// <summary> Time saving in <see cref="TimeSpan"/> format in effect at the <c>ReferenceUTCTimestamp</c>. </summary>
public TimeSpan DaylightSavingsTimeSpan { get; }
public TimeSpan DaylightSavingsOffset { get; }

/// <summary> Current wall time at the given time zone as shown in the <c>Tag</c> property. </summary>
[CodeGenMember("WallTime")]
Expand Down
4 changes: 2 additions & 2 deletions sdk/maps/Azure.Maps.TimeZones/src/Models/TimeTransition.cs
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ internal TimeTransition(string tag, string standardOffset, string daylightSaving
_daylightSavings = daylightSavings;
if (TimeSpan.TryParse(daylightSavings, out TimeSpan DaylightSavingsValue))
{
DaylightSavingsTimeSpan = DaylightSavingsValue;
DaylightSavingsOffset = DaylightSavingsValue;
}

UtcStart = utcStart;
Expand All @@ -45,6 +45,6 @@ internal TimeTransition(string tag, string standardOffset, string daylightSaving
[CodeGenMember("DaylightSavings")]
internal string _daylightSavings { get; }
/// <summary> Time saving in <see cref="TimeSpan"/> format in effect at the <c>ReferenceUTCTimestamp</c>. </summary>
public TimeSpan DaylightSavingsTimeSpan { get; }
public TimeSpan DaylightSavingsOffset { get; }
}
}

0 comments on commit 529e8fa

Please sign in to comment.