Bug fix

src/Inc.TeamAssistant.Appraiser.Application/QueryHandlers/GetStoryDetails/GetStoryDetailsQueryHandler.cs

@@ -48,9 +48,9 @@ private GetStoryDetailsResult Get(Story story)
 
         return new(
 	        story.Title,
+	        story.Links.ToArray(),
             code,
             StorySelected: story != Story.Empty,
-            new StoryDetails(story.Title, story.Links.ToArray()),
             items,
             story.GetTotal().ToDisplayValue(estimateEnded, story.StoryType));
     }

src/Inc.TeamAssistant.Appraiser.Model/IAssessmentSessionsService.cs → src/Inc.TeamAssistant.Appraiser.Model/IAppraiserService.cs

@@ -4,10 +4,10 @@
 
 namespace Inc.TeamAssistant.Appraiser.Model;
 
-public interface IAssessmentSessionsService
+public interface IAppraiserService
 {
 	Task<ServiceResult<GetStoryDetailsResult?>> GetStoryDetails(
-        Guid assessmentSessionId,
+        Guid teamId,
 		CancellationToken cancellationToken = default);
 
     Task<ServiceResult<GetLinkForConnectResult>> GetLinkForConnect(CancellationToken cancellationToken = default);

src/Inc.TeamAssistant.Appraiser.Model/IEventsProvider.cs

@@ -2,5 +2,5 @@ namespace Inc.TeamAssistant.Appraiser.Model;
 
 public interface IEventsProvider : IAsyncDisposable
 {
-    Task OnStoryChanged(Guid assessmentSessionId, Func<Task> changed);
+    Task OnStoryChanged(Guid teamId, Func<Task> changed);
 }

src/Inc.TeamAssistant.Appraiser.Model/Queries/GetStoryDetails/GetStoryDetailsResult.cs

@@ -1,9 +1,9 @@
 namespace Inc.TeamAssistant.Appraiser.Model.Queries.GetStoryDetails;
 
 public sealed record GetStoryDetailsResult(
-	string AssessmentSessionTitle,
+	string Title,
+	IReadOnlyCollection<string> Links,
     string CodeForConnect,
     bool StorySelected,
-    StoryDetails Story,
 	IReadOnlyCollection<StoryForEstimateDto> StoryForEstimates,
 	string Total);

src/Inc.TeamAssistant.Connector.Application/CommandHandlers/CreateTeam/Validators/CreateTeamCommandValidator.cs

@@ -12,6 +12,7 @@ public CreateTeamCommandValidator()
 
         RuleFor(e => e.Name)
             .NotEmpty()
+            .MaximumLength(255)
             .Must(e => !e.StartsWith("/"))
             .WithMessage("Please enter text value.");
     }

src/Inc.TeamAssistant.Connector.Application/Inc.TeamAssistant.Connector.Application.csproj

@@ -3,6 +3,7 @@
     <PackageReference Include="FluentValidation" Version="11.7.1" />
     <PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" Version="7.0.0" />
     <PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="7.0.1" />
+    <PackageReference Include="Npgsql" Version="7.0.6" />
     <PackageReference Include="Telegram.Bot" Version="19.0.0" />
   </ItemGroup>
   <ItemGroup>

src/Inc.TeamAssistant.Connector.Application/Services/TelegramBotMessageHandler.cs

@@ -6,6 +6,7 @@
 using MediatR;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Logging;
+using Npgsql;
 using Telegram.Bot;
 using Telegram.Bot.Types;
 using Telegram.Bot.Types.ReplyMarkups;
@@ -45,6 +46,7 @@ public async Task Handle(ITelegramBotClient client, Update update, Guid botId, C
         if (update is null)
             throw new ArgumentNullException(nameof(update));
 
+        const string duplicateKeyError = "23505";
         var bot = await _botRepository.Find(botId, token);
         if (bot is null)
             throw new TeamAssistantUserException(Messages.Connector_BotNotFound, botId);
@@ -69,9 +71,17 @@ public async Task Handle(ITelegramBotClient client, Update update, Guid botId, C
                 userException.MessageId,
                 messageContext.LanguageId,
                 userException.Values);
-            
+
             await TrySend(client, messageContext.ChatId, errorMessage, token);
         }
+        catch (PostgresException ex) when (ex.SqlState == duplicateKeyError)
+        {
+            await TrySend(
+                client,
+                messageContext.ChatId,
+                "Duplicate key value violates unique constraint.",
+                token);
+        }
         catch (Exception ex)
         {
             _logger.LogError(ex, "Unhandled exception");

src/Inc.TeamAssistant.Connector.DataAccess/Inc.TeamAssistant.Connector.DataAccess.csproj

@@ -4,6 +4,6 @@
   </ItemGroup>
   <ItemGroup>
     <PackageReference Include="Dapper" Version="2.1.28" />
-    <PackageReference Include="Npgsql" Version="8.0.1" />
+    <PackageReference Include="Npgsql" Version="7.0.6" />
   </ItemGroup>
 </Project>

src/Inc.TeamAssistant.Connector.DataAccess/TeamRepository.cs

@@ -63,7 +63,7 @@ public async Task Upsert(Team team, CancellationToken token)
 
         var upsertTeam = new CommandDefinition(@"
             INSERT INTO connector.teams (id, bot_id, chat_id, name, properties)
-            VALUES (@id, @bot_id, @chat_id, @name, @properties)
+            VALUES (@id, @bot_id, @chat_id, @name, @properties::jsonb)
             ON CONFLICT (id) DO UPDATE SET
                 bot_id = EXCLUDED.bot_id,
                 chat_id = EXCLUDED.chat_id,

src/Inc.TeamAssistant.Gateway/Controllers/AssessmentSessionsController.cs → src/Inc.TeamAssistant.Gateway/Controllers/SessionsController.cs

@@ -5,18 +5,18 @@ namespace Inc.TeamAssistant.Gateway.Controllers;
 
 [ApiController]
 [Route("sessions")]
-public sealed class AssessmentSessionsController : ControllerBase
+public sealed class SessionsController : ControllerBase
 {
-	private readonly IAssessmentSessionsService _service;
+	private readonly IAppraiserService _service;
 
-    public AssessmentSessionsController(IAssessmentSessionsService service)
+    public SessionsController(IAppraiserService service)
     {
         _service = service ?? throw new ArgumentNullException(nameof(service));
     }
 
-    [HttpGet("story/{assessmentSessionId}")]
-	public async Task<IActionResult> GetStoryDetails(Guid assessmentSessionId, CancellationToken cancellationToken)
-		=> Ok(await _service.GetStoryDetails(assessmentSessionId, cancellationToken));
+    [HttpGet("story/{teamId}")]
+	public async Task<IActionResult> GetStoryDetails(Guid teamId, CancellationToken cancellationToken)
+		=> Ok(await _service.GetStoryDetails(teamId, cancellationToken));
 
     [HttpGet("link-for-connect")]
     public async Task<IActionResult> LinkForConnect(CancellationToken cancellationToken)

src/Inc.TeamAssistant.Gateway/Services/AssessmentSessionsService.cs → src/Inc.TeamAssistant.Gateway/Services/AppraiserService.cs

@@ -6,21 +6,21 @@
 
 namespace Inc.TeamAssistant.Gateway.Services;
 
-internal sealed class AssessmentSessionsService : IAssessmentSessionsService
+internal sealed class AppraiserService : IAppraiserService
 {
 	private readonly IMediator _mediator;
 
-	public AssessmentSessionsService(IMediator mediator)
+	public AppraiserService(IMediator mediator)
 		=> _mediator = mediator ?? throw new ArgumentNullException(nameof(mediator));
 
 	public async Task<ServiceResult<GetStoryDetailsResult?>> GetStoryDetails(
-        Guid assessmentSessionId,
+        Guid teamId,
 		CancellationToken cancellationToken)
 	{
 		try
 		{
 			var result = await _mediator.Send(
-                new GetStoryDetailsQuery(assessmentSessionId),
+                new GetStoryDetailsQuery(teamId),
                 cancellationToken);
 
 			return result is null

src/Inc.TeamAssistant.Gateway/Services/EventsProvider.cs

@@ -4,7 +4,7 @@ namespace Inc.TeamAssistant.Gateway.Services;
 
 internal sealed class EventsProvider : IEventsProvider
 {
-    public Task OnStoryChanged(Guid assessmentSessionId, Func<Task> changed) => Task.CompletedTask;
+    public Task OnStoryChanged(Guid teamId, Func<Task> changed) => Task.CompletedTask;
 
     public ValueTask DisposeAsync() => ValueTask.CompletedTask;
 }

src/Inc.TeamAssistant.Gateway/Services/ServiceCollectionExtensions.cs

@@ -31,7 +31,7 @@ public static IServiceCollection AddServices(
                 options.CacheAbsoluteExpiration));
 
         services
-            .AddScoped<IAssessmentSessionsService, AssessmentSessionsService>()
+            .AddScoped<IAppraiserService, AppraiserService>()
             .AddSingleton<IEventsProvider, EventsProvider>()
             .AddScoped<ICookieService, CookieService>()
             .AddScoped<IMessagesSender, MessagesSender>()

src/Inc.TeamAssistant.Gateway/appsettings.json

@@ -7,7 +7,7 @@
     }
   },
   "AllowedHosts": "*",
-  "AddStoryToAssessmentSessionOptions": {
+  "AddStoryOptions": {
     "LinksPrefix": ["http://", "https://"]
   },
   "TelegramBotOptions": {

src/Inc.TeamAssistant.Migrations/2024_01_03_1_CreateConnectorTables.cs

@@ -17,6 +17,7 @@ public override void Up()
 
             .WithColumn("name")
             .AsString(50).NotNullable()
+            .Unique("bots__uidx__name")
 
             .WithColumn("token")
             .AsString(255).NotNullable();
@@ -62,6 +63,10 @@ public override void Up()
 
             .WithColumn("properties")
             .AsCustom("jsonb").NotNullable();
+
+        Execute.Sql(
+            "CREATE UNIQUE INDEX teams__uidx__bot_id__name ON connector.teams (bot_id, lower(name));",
+            "Create index by team name");
 
         Create
             .Table("teammates")

src/Inc.TeamAssistant.Migrations/2024_01_11_0_ChangeReviewerTables.cs

@@ -61,6 +61,13 @@ FROM review.players
             .InSchema("review")
 
             .AsInt32().NotNullable().SetExistingRowsTo(1);
+
+        Alter
+            .Column("description")
+            .OnTable("task_for_reviews")
+            .InSchema("review")
+            .AsString(2000)
+            .NotNullable();
     }
 
     public override void Down()

src/Inc.TeamAssistant.Reviewer.Application/CommandHandlers/MoveToReview/Validators/MoveToReviewCommandValidator.cs

@@ -12,6 +12,7 @@ public MoveToReviewCommandValidator()
 
         RuleFor(e => e.Description)
             .NotEmpty()
+            .MaximumLength(2000)
             .Must(e => !e.StartsWith("/"))
             .WithMessage("Please enter text value.");
     }

src/Inc.TeamAssistant.WebUI/Components/About.razor

@@ -2,7 +2,7 @@
 @using Inc.TeamAssistant.Appraiser.Model.Common
 @using Inc.TeamAssistant.WebUI.Components.Annotations
 
-@inject IAssessmentSessionsService Service
+@inject IAppraiserService Service
 @inject LanguageManager LanguageManager
 
 <div class="about__body">

src/Inc.TeamAssistant.WebUI/Pages/AssessmentSession.razor

@@ -5,7 +5,7 @@
 @using Inc.TeamAssistant.Appraiser.Model.Common
 @using Inc.TeamAssistant.WebUI.Pages.Annotations
 
-@inject IAssessmentSessionsService Service
+@inject IAppraiserService Service
 @inject IEventsProvider EventsProvider
 @inject LanguageManager LanguageManager
 
@@ -18,16 +18,14 @@
                 @if (_model is { State: ServiceResultState.Success, Result: not null })
                 {
                     <div class="assessment-session__details">
-                        <span class="assessment-session__subtitle">@_model.Result.AssessmentSessionTitle</span>
-
                         @if (_model.Result.StorySelected)
                         {
-                            <h1 class="assessment-session__title">@_annotation.GetTitle(_model.Result.Story.Title)</h1>
+                            <h1 class="assessment-session__title">@_annotation.GetTitle(_model.Result.Title)</h1>
 
-                            if (_model.Result.Story.Links.Any())
+                            if (_model.Result.Links.Any())
                             {
                                 <ul class="assessment-session__items">
-                                    @foreach (var link in _model.Result.Story.Links)
+                                    @foreach (var link in _model.Result.Links)
                                     {
                                         <li class="assessment-session__item">
                                             <a class="assessment-session__link" href="@link" target="_blank">@link</a>
@@ -89,7 +87,7 @@
     private AssessmentSessionAnnotation _annotation = AssessmentSessionAnnotation.Empty;
 
     private MarkupString PageTitle => _model is { State: ServiceResultState.Success, Result: not null }
-        ? (MarkupString)_model.Result.AssessmentSessionTitle
+        ? (MarkupString)_model.Result.Title
         : _annotation.AssessmentSessionCard;
 
     private MarkupString StateMessage => _model.State switch

src/Inc.TeamAssistant.WebUI/Pages/AssessmentSession.razor.css

@@ -18,10 +18,6 @@
     margin: 0 0 .75rem 0;
 	letter-spacing: .0625rem;
 }
-.assessment-session__subtitle {
-    margin: 0 0 .75rem 0;
-    display: block;
-}
 .assessment-session__text {
 	line-height: 1.6rem;
 }

src/Inc.TeamAssistant.WebUI/Services/AssessmentSessionsClient.cs → src/Inc.TeamAssistant.WebUI/Services/AppraiserClient.cs

@@ -7,23 +7,23 @@
 
 namespace Inc.TeamAssistant.WebUI.Services;
 
-internal sealed class AssessmentSessionsClient : IAssessmentSessionsService
+internal sealed class AppraiserClient : IAppraiserService
 {
 	private readonly HttpClient _client;
 
-    public AssessmentSessionsClient(HttpClient client)
+    public AppraiserClient(HttpClient client)
     {
         _client = client ?? throw new ArgumentNullException(nameof(client));
     }
 
 	public async Task<ServiceResult<GetStoryDetailsResult?>> GetStoryDetails(
-        Guid assessmentSessionId,
+        Guid teamId,
 		CancellationToken cancellationToken)
 	{
 		try
 		{
 			var result = await _client.GetFromJsonAsync<ServiceResult<GetStoryDetailsResult?>>(
-				$"sessions/story/{assessmentSessionId}",
+				$"sessions/story/{teamId}",
 				cancellationToken);
 
 			if (result is null)

src/Inc.TeamAssistant.WebUI/Services/EventsProviderClient.cs

@@ -19,15 +19,15 @@ public EventsProviderClient(NavigationManager navigationManager)
             .Build();
 	}
 
-	public async Task OnStoryChanged(Guid assessmentSessionId, Func<Task> changed)
+	public async Task OnStoryChanged(Guid teamId, Func<Task> changed)
 	{
 		if (changed is null)
 			throw new ArgumentNullException(nameof(changed));
 
         _hubConnection.On("StoryChanged", changed);
 
         await _hubConnection.StartAsync();
-        await _hubConnection.InvokeAsync("JoinToGroup", assessmentSessionId);
+        await _hubConnection.InvokeAsync("JoinToGroup", teamId);
     }
 
     public ValueTask DisposeAsync() => _hubConnection.DisposeAsync();

src/Inc.TeamAssistant.WebUI/Services/ServiceCollectionExtensions.cs

@@ -18,7 +18,7 @@ internal static IServiceCollection AddServices(this IServiceCollection services)
         services
             .AddBlazoredLocalStorage()
 
-            .AddScoped<IAssessmentSessionsService, AssessmentSessionsClient>()
+            .AddScoped<IAppraiserService, AppraiserClient>()
             .AddScoped<IClientInfoService, ClientInfoClient>()
             .AddSingleton<IEventsProvider, EventsProviderClient>()
             .AddSingleton<ICookieService, CookieServiceClient>()