The creation time of workflow is not intuitive enough

[hyzx86]

In the current list of workflow instances, the process creation time is not intuitive enough. We only can open a detailed page to check the exact execution time

I adjusted the time format of the list page to make it consistent with the details page

@Skrypt
If necessary, I can submit a PR，Or you can update directly with the following code

<span class="hint">@T["Created {0}", await LocalClock.ConvertToLocalAsync(entry.Workflow.CreatedUtc)]</span>  

[hishamco]

Do we really show time zone, coz this is the first time I notice that ;)

[jtkech]

Here the "concern" is the usage by design of our coded TimeSpan shape

OrchardCore/src/OrchardCore/OrchardCore.DisplayManagement/Shapes/DateTimeShapes.cs

Lines 35 to 46 in c255456

	[Shape]
	public IHtmlContent TimeSpan(IHtmlHelper Html, DateTime? Utc, DateTime? Origin)
	{
	Utc = Utc ?? _clock.UtcNow;
	Origin = Origin ?? _clock.UtcNow;
	var time = _clock.UtcNow - Utc.Value;
	if (time.TotalYears() > 1)
	return H.Plural(time.TotalYears(), "1 year ago", "{0} years ago");
	if (time.TotalYears() < -1)
	return H.Plural(-time.TotalYears(), "in 1 year", "in {0} years");

@hyzx86 would prefer the usage of our DateTime shape whose code looks like his suggestion ;)

OrchardCore/src/OrchardCore/OrchardCore.DisplayManagement/Shapes/DateTimeShapes.cs

Lines 83 to 87 in c255456

	[Shape]
	public async Task<IHtmlContent> DateTime(IHtmlHelper Html, DateTime? Utc, string Format)
	{
	Utc = Utc ?? _clock.UtcNow;
	var zonedTime = await _localClock.ConvertToLocalAsync(Utc.Value);

[hyzx86]

🤣This is the first time I use shape..

Refer to official documents https://docs.orchardcore.net/en/dev/docs/reference/core/Placement/#datetime

I use it this way, I don't konw why it is empty ..
this code is in my custom theme project

[hishamco]

You might need to override the shape

[hyzx86]

Done ,but that will be show a correct time , 8 hours missing..

Looks like the Format is not work

 <span class="hint">@T["Created"]@await DisplayAsync(await New.DateTime(entry.Workflow.CreatedUtc, Format: "yyyy-MM-dd hh:mm:ss"))</span>
 @* <span class="hint">@T["Created: {0}", await LocalClock.ConvertToLocalAsync(entry.Workflow.CreatedUtc)]</span> *@
 @if (entry.Workflow.State.ContainsKey("LastExecutedOn")
      && entry.Workflow.State["LastExecutedOn"] is not null
      && DateTime.TryParse(entry.Workflow.State["LastExecutedOn"].ToString(), out var lastExecutedOn))
 {
 lastExecutedOn = DateTime.SpecifyKind(lastExecutedOn, DateTimeKind.Utc);
 <span class="hint">@T["| Last Executed: {0}", await LocalClock.ConvertToLocalAsync(lastExecutedOn)]</span>
 }

[hishamco]

@hyzx86 why you didn't set a proper time zone

[hyzx86]

In my orginal code ，it is work well

<span class="hint">@T["Created {0}", await LocalClock.ConvertToLocalAsync(entry.Workflow.CreatedUtc)]</span>  

The following code also uses ConvertToLocalAsync. I think it should output the same or similar results, but this is not the case

OrchardCore/src/OrchardCore/OrchardCore.DisplayManagement/Shapes/DateTimeShapes.cs

Lines 83 to 94 in c255456

	[Shape]
	public async Task<IHtmlContent> DateTime(IHtmlHelper Html, DateTime? Utc, string Format)
	{
	Utc = Utc ?? _clock.UtcNow;
	var zonedTime = await _localClock.ConvertToLocalAsync(Utc.Value);
	if (Format == null)
	{
	Format = S[LongDateTimeFormat].Value;
	}
	return Html.Raw(Html.Encode(zonedTime.ToString(Format, CultureInfo.CurrentUICulture)));

[hishamco]

What you wanna undo?

[sebastienros]

Suggestion:

Change this line:

https://github.com/OrchardCMS/OrchardCore/blob/main/src/OrchardCore.Modules/OrchardCore.Workflows/Views/Workflow/Index.cshtml#L104

With

<span class="hint">@T["Created {0}", (object)(await DisplayAsync(await New.DateTime(Utc: entry.Workflow.CreatedUtc)))]</span>