Browse Source

minor ui fixes (#668)

* remove unused package

* fix controller name

* catch and ignore jsruntime exceptions

* separate debug stage from public stage
pull/669/head
Jason Dove 4 years ago committed by GitHub
parent
commit
2058c44949
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 4
      ErsatzTV/Controllers/ArtworkController.cs
  2. 1
      ErsatzTV/ErsatzTV.csproj
  3. 8
      ErsatzTV/Pages/CollectionItems.razor
  4. 7
      ErsatzTV/Pages/FragmentNavigationBase.cs
  5. 12
      ErsatzTV/Pages/PlexMediaSources.razor
  6. 7
      ErsatzTV/Pages/TelevisionEpisodeList.razor
  7. 13
      ErsatzTV/Shared/MarkdownView.razor.cs
  8. 8
      ErsatzTV/Startup.cs

4
ErsatzTV/Controllers/ArtworkController.cs

@ -25,12 +25,12 @@ namespace ErsatzTV.Controllers
[ResponseCache(Duration = 3600)] [ResponseCache(Duration = 3600)]
[ApiController] [ApiController]
[ApiExplorerSettings(IgnoreApi = true)] [ApiExplorerSettings(IgnoreApi = true)]
public class PostersController : ControllerBase public class ArtworkController : ControllerBase
{ {
private readonly IHttpClientFactory _httpClientFactory; private readonly IHttpClientFactory _httpClientFactory;
private readonly IMediator _mediator; private readonly IMediator _mediator;
public PostersController(IMediator mediator, IHttpClientFactory httpClientFactory) public ArtworkController(IMediator mediator, IHttpClientFactory httpClientFactory)
{ {
_mediator = mediator; _mediator = mediator;
_httpClientFactory = httpClientFactory; _httpClientFactory = httpClientFactory;

1
ErsatzTV/ErsatzTV.csproj

@ -11,7 +11,6 @@
</ItemGroup> </ItemGroup>
<ItemGroup> <ItemGroup>
<PackageReference Include="Blazored.LocalStorage" Version="4.2.0" />
<PackageReference Include="Bugsnag.AspNet.Core" Version="3.0.0" /> <PackageReference Include="Bugsnag.AspNet.Core" Version="3.0.0" />
<PackageReference Include="FluentValidation" Version="10.3.6" /> <PackageReference Include="FluentValidation" Version="10.3.6" />
<PackageReference Include="FluentValidation.AspNetCore" Version="10.3.6" /> <PackageReference Include="FluentValidation.AspNetCore" Version="10.3.6" />

8
ErsatzTV/Pages/CollectionItems.razor

@ -312,6 +312,8 @@
} }
protected override async Task OnAfterRenderAsync(bool firstRender) protected override async Task OnAfterRenderAsync(bool firstRender)
{
try
{ {
await _jsRuntime.InvokeVoidAsync("sortableCollection", Id); await _jsRuntime.InvokeVoidAsync("sortableCollection", Id);
if (_data.UseCustomPlaybackOrder) if (_data.UseCustomPlaybackOrder)
@ -322,6 +324,12 @@
{ {
await _jsRuntime.InvokeVoidAsync("disableSorting"); await _jsRuntime.InvokeVoidAsync("disableSorting");
} }
}
catch (Exception)
{
// ignored
}
await base.OnAfterRenderAsync(firstRender); await base.OnAfterRenderAsync(firstRender);
} }

7
ErsatzTV/Pages/FragmentNavigationBase.cs

@ -34,9 +34,16 @@ namespace ErsatzTV.Pages
protected override async Task OnAfterRenderAsync(bool firstRender) protected override async Task OnAfterRenderAsync(bool firstRender)
{ {
if (firstRender) if (firstRender)
{
try
{ {
await NavManager.NavigateToFragmentAsync(JsRuntime); await NavManager.NavigateToFragmentAsync(JsRuntime);
} }
catch (Exception)
{
// ignored
}
}
} }
[SuppressMessage("ReSharper", "VSTHRD100")] [SuppressMessage("ReSharper", "VSTHRD100")]

12
ErsatzTV/Pages/PlexMediaSources.razor

@ -125,7 +125,17 @@
{ {
Either<BaseError, string> maybeUrl = await _mediator.Send(new StartPlexPinFlow()); Either<BaseError, string> maybeUrl = await _mediator.Send(new StartPlexPinFlow());
await maybeUrl.Match( await maybeUrl.Match(
async url => await _jsRuntime.InvokeAsync<object>("open", new object[] { url, "_blank" }), async url =>
{
try
{
await _jsRuntime.InvokeAsync<object>("open", new object[] { url, "_blank" });
}
catch (Exception)
{
// ignored
}
},
error => error =>
{ {
_locker.UnlockPlex(); _locker.UnlockPlex();

7
ErsatzTV/Pages/TelevisionEpisodeList.razor

@ -179,9 +179,16 @@
protected override async Task OnAfterRenderAsync(bool firstRender) protected override async Task OnAfterRenderAsync(bool firstRender)
{ {
if (firstRender) if (firstRender)
{
try
{ {
await _navigationManager.NavigateToFragmentAsync(_jsRuntime); await _navigationManager.NavigateToFragmentAsync(_jsRuntime);
} }
catch (Exception)
{
// ignored
}
}
} }
protected override Task OnParametersSetAsync() => RefreshData(); protected override Task OnParametersSetAsync() => RefreshData();

13
ErsatzTV/Shared/MarkdownView.razor.cs

@ -1,4 +1,5 @@
using System.Threading.Tasks; using System;
using System.Threading.Tasks;
using Ganss.XSS; using Ganss.XSS;
using Markdig; using Markdig;
using Microsoft.AspNetCore.Components; using Microsoft.AspNetCore.Components;
@ -47,9 +48,17 @@ namespace ErsatzTV.Shared
} }
protected override async Task OnAfterRenderAsync(bool firstRender) protected override async Task OnAfterRenderAsync(bool firstRender)
{
try
{ {
await JsRuntime.InvokeVoidAsync("styleMarkdown"); await JsRuntime.InvokeVoidAsync("styleMarkdown");
await base.OnAfterRenderAsync(firstRender); }
catch (Exception)
{
// ignored
}
await base.OnAfterRenderAsync(true);
} }
} }
} }

8
ErsatzTV/Startup.cs

@ -4,7 +4,6 @@ using System.IO;
using System.Reflection; using System.Reflection;
using System.Text; using System.Text;
using System.Threading.Channels; using System.Threading.Channels;
using Blazored.LocalStorage;
using Bugsnag.AspNet.Core; using Bugsnag.AspNet.Core;
using Dapper; using Dapper;
using ErsatzTV.Application; using ErsatzTV.Application;
@ -94,10 +93,14 @@ namespace ErsatzTV
?.GetCustomAttribute<AssemblyInformationalVersionAttribute>() ?.GetCustomAttribute<AssemblyInformationalVersionAttribute>()
?.InformationalVersion ?? "unknown"; ?.InformationalVersion ?? "unknown";
configuration.NotifyReleaseStages = new[] { "public" }; configuration.NotifyReleaseStages = new[] { "public", "develop" };
#if DEBUG
configuration.ReleaseStage = "develop";
#else
// effectively "disable" by tweaking app config // effectively "disable" by tweaking app config
configuration.ReleaseStage = bugsnagConfig.Enable ? "public" : "private"; configuration.ReleaseStage = bugsnagConfig.Enable ? "public" : "private";
#endif
}); });
services.AddCors( services.AddCors(
@ -139,7 +142,6 @@ namespace ErsatzTV
services.AddMudServices(); services.AddMudServices();
services.AddCourier(Assembly.GetAssembly(typeof(LibraryScanProgress))); services.AddCourier(Assembly.GetAssembly(typeof(LibraryScanProgress)));
services.AddBlazoredLocalStorage();
Console.OutputEncoding = Encoding.UTF8; Console.OutputEncoding = Encoding.UTF8;

Loading…
Cancel
Save