Skip to content

Commit

Permalink
feat(offers): remove provider from offer (#911)
Browse files Browse the repository at this point in the history
* feat(offers): remove provider from offer
instead of having a separate property for the provider the providing companies name is used
* change offer provider_company_id to mandatory (non-nullable)
Refs: #811
---------
Co-authored-by: Norbert Truchsess <norbert.truchsess@t-online.de>
  • Loading branch information
Phil91 authored Sep 12, 2024
1 parent 444a8ab commit 4e3958c
Show file tree
Hide file tree
Showing 54 changed files with 10,432 additions and 600 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@
* SPDX-License-Identifier: Apache-2.0
********************************************************************************/

using Org.Eclipse.TractusX.Portal.Backend.Administration.Service.Models;
using Org.Eclipse.TractusX.Portal.Backend.PortalBackend.DBAccess.Models;

namespace Org.Eclipse.TractusX.Portal.Backend.Administration.Service.BusinessLogic;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
********************************************************************************/

using Microsoft.Extensions.Options;
using Org.Eclipse.TractusX.Portal.Backend.Administration.Service.Models;
using Org.Eclipse.TractusX.Portal.Backend.Framework.Async;
using Org.Eclipse.TractusX.Portal.Backend.Framework.ErrorHandling;
using Org.Eclipse.TractusX.Portal.Backend.Framework.Linq;
Expand Down
4 changes: 0 additions & 4 deletions src/externalsystems/Custodian.Library/ICustodianService.cs
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,6 @@

using Org.Eclipse.TractusX.Portal.Backend.Custodian.Library.Models;
using Org.Eclipse.TractusX.Portal.Backend.Framework.ErrorHandling;
using Org.Eclipse.TractusX.Portal.Backend.PortalBackend.DBAccess.Models;
using Org.Eclipse.TractusX.Portal.Backend.PortalBackend.PortalEntities.Enums;
using System.Threading;
using System.Threading.Tasks;

namespace Org.Eclipse.TractusX.Portal.Backend.Custodian.Library;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@
using Org.Eclipse.TractusX.Portal.Backend.IssuerComponent.Library.Models;
using Org.Eclipse.TractusX.Portal.Backend.IssuerComponent.Library.Service;
using Org.Eclipse.TractusX.Portal.Backend.PortalBackend.DBAccess;
using Org.Eclipse.TractusX.Portal.Backend.PortalBackend.DBAccess.Models;
using Org.Eclipse.TractusX.Portal.Backend.PortalBackend.DBAccess.Repositories;
using Org.Eclipse.TractusX.Portal.Backend.PortalBackend.PortalEntities.Enums;
using Org.Eclipse.TractusX.Portal.Backend.Processes.ApplicationChecklist.Library;
Expand Down
1 change: 0 additions & 1 deletion src/keycloak/Keycloak.Factory/KeycloakSettingData.cs
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@
using Microsoft.Extensions.Configuration;
using Microsoft.Extensions.DependencyInjection;
using Org.Eclipse.TractusX.Portal.Backend.Framework.ErrorHandling;
using System.Runtime.Serialization;

namespace Org.Eclipse.TractusX.Portal.Backend.Keycloak.Factory;

Expand Down
1 change: 0 additions & 1 deletion src/mailing/Mailing.SendMail/MailSettings.cs
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@

using Microsoft.Extensions.Configuration;
using Microsoft.Extensions.DependencyInjection;
using MimeKit;
using Org.Eclipse.TractusX.Portal.Backend.Framework.ErrorHandling;

namespace Org.Eclipse.TractusX.Portal.Backend.Mailing.SendMail
Expand Down
192 changes: 83 additions & 109 deletions src/marketplace/Apps.Service/BusinessLogic/AppReleaseBusinessLogic.cs

Large diffs are not rendered by default.

3 changes: 0 additions & 3 deletions src/marketplace/Apps.Service/ViewModels/AppRequestModel.cs
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
/********************************************************************************
* Copyright (c) 2022 BMW Group AG
* Copyright (c) 2022 Contributors to the Eclipse Foundation
*
* See the NOTICE file(s) distributed with this work for additional
Expand Down Expand Up @@ -28,7 +27,6 @@ namespace Org.Eclipse.TractusX.Portal.Backend.Apps.Service.ViewModels;
/// Request Model for App Creation.
/// </summary>
/// <param name="Title">Title</param>
/// <param name="Provider">Provider</param>
/// <param name="SalesManagerId">SalesManagerId</param>
/// <param name="UseCaseIds">UseCaseIds</param>
/// <param name="Descriptions">Descriptions</param>
Expand All @@ -40,7 +38,6 @@ namespace Org.Eclipse.TractusX.Portal.Backend.Apps.Service.ViewModels;
/// <param name="ContactNumber">Price</param>
public record AppRequestModel(
string? Title,
string Provider,
Guid? SalesManagerId,
IEnumerable<Guid> UseCaseIds,
IEnumerable<LocalizedDescription> Descriptions,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
* SPDX-License-Identifier: Apache-2.0
********************************************************************************/

using Microsoft.Extensions.Configuration;
using Microsoft.Extensions.DependencyInjection;
using Org.Eclipse.TractusX.Portal.Backend.Framework.HttpClientExtensions;
using Org.Eclipse.TractusX.Portal.Backend.Framework.Token;
Expand Down
178 changes: 71 additions & 107 deletions src/marketplace/Offers.Library/Service/OfferService.cs

Large diffs are not rendered by default.

Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
********************************************************************************/

using Org.Eclipse.TractusX.Portal.Backend.PortalBackend.PortalEntities.Enums;
using System.Reflection.Metadata;

namespace Org.Eclipse.TractusX.Portal.Backend.PortalBackend.DBAccess.Models;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
/********************************************************************************
* Copyright (c) 2022 BMW Group AG
* Copyright (c) 2022 Contributors to the Eclipse Foundation
*
* See the NOTICE file(s) distributed with this work for additional
Expand All @@ -25,7 +24,6 @@ namespace Org.Eclipse.TractusX.Portal.Backend.PortalBackend.DBAccess.Models;
/// Data Object for the Offer Release data
/// </summary>
/// <param name="Name"></param>
/// <param name="ProviderCompanyId"></param>
/// <param name="CompanyName"></param>
/// <param name="IsDescriptionLongNotSet"></param>
/// <param name="IsDescriptionShortNotSet"></param>
Expand All @@ -35,7 +33,6 @@ namespace Org.Eclipse.TractusX.Portal.Backend.PortalBackend.DBAccess.Models;
/// <returns></returns>
public record OfferReleaseData(
string? Name,
Guid? ProviderCompanyId,
string CompanyName,
bool IsDescriptionLongNotSet,
bool IsDescriptionShortNotSet,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
/********************************************************************************
* Copyright (c) 2022 BMW Group AG
* Copyright (c) 2022 Contributors to the Eclipse Foundation
*
* See the NOTICE file(s) distributed with this work for additional
Expand Down Expand Up @@ -36,7 +35,7 @@ namespace Org.Eclipse.TractusX.Portal.Backend.PortalBackend.DBAccess.Models;
/// <param name="ServiceTypeIds">Collection of the assigned serviceTypeIds.</param>
public record ServiceOverviewData(
Guid Id,
string Title,
string? Title,
string Provider,
string? ContactEmail,
string? Description,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -83,12 +83,6 @@ public void CreateUpdateDeleteIdentifiers(Guid companyId, IEnumerable<(UniqueIde
(entity, initial) => entity.Value = initial.Value,
(entity, modified) => entity.Value = modified.Value);

public Task<(bool IsValidCompany, string CompanyName)> GetCompanyNameUntrackedAsync(Guid companyId) =>
context.Companies
.Where(x => x.Id == companyId)
.Select(company => new ValueTuple<bool, string>(true, company.Name))
.SingleOrDefaultAsync();

public Task<(string? Bpn, IEnumerable<Guid> TechnicalUserRoleIds)> GetBpnAndTechnicalUserRoleIds(Guid companyId, string technicalUserClientId) =>
context.Companies
.AsNoTracking()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@

using Org.Eclipse.TractusX.Portal.Backend.Framework.Models;
using Org.Eclipse.TractusX.Portal.Backend.PortalBackend.DBAccess.Models;
using Org.Eclipse.TractusX.Portal.Backend.PortalBackend.PortalEntities;
using Org.Eclipse.TractusX.Portal.Backend.PortalBackend.PortalEntities.Entities;
using Org.Eclipse.TractusX.Portal.Backend.PortalBackend.PortalEntities.Enums;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,6 @@ public interface ICompanyRepository

void CreateUpdateDeleteIdentifiers(Guid companyId, IEnumerable<(UniqueIdentifierId UniqueIdentifierId, string Value)> initialItems, IEnumerable<(UniqueIdentifierId UniqueIdentifierId, string Value)> modifiedItems);

Task<(bool IsValidCompany, string CompanyName)> GetCompanyNameUntrackedAsync(Guid companyId);

Task<(string? Bpn, IEnumerable<Guid> TechnicalUserRoleIds)> GetBpnAndTechnicalUserRoleIds(Guid companyId, string technicalUserClientId);

/// <summary>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,10 @@ public interface IOfferRepository
/// <summary>
/// Adds an app to the database
/// </summary>
/// <param name="provider">Provider of the app</param>
/// <param name="offerType">Type of the app</param>
/// <param name="providerCompanyId">the offer providers CompanyId</param>
/// <param name="setOptionalParameters">Action to set the optional parameters</param>
Offer CreateOffer(string provider, OfferTypeId offerType, Action<Offer>? setOptionalParameters = null);
Offer CreateOffer(OfferTypeId offerType, Guid providerCompanyId, Action<Offer>? setOptionalParameters = null);

void AttachAndModifyOffer(Guid offerId, Action<Offer> setOptionalParameters, Action<Offer>? initializeParemeters = null);

Expand Down
Loading

0 comments on commit 4e3958c

Please sign in to comment.