Skip to content

Commit

Permalink
Admin UI: Tier quotas don't show up after changing the tier of an ide…
Browse files Browse the repository at this point in the history
…ntity (#678)

* fix: give TierOfIdentityChangedDomainEvent a unique id

* chore: fix namespace

* chore: give DatawalletModifiedDomainEvent a descriptive id

* chore: copy default value for subscriptionClientName to helm values
  • Loading branch information
tnotheis authored May 29, 2024
1 parent 7c96ffd commit b40710e
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 5 deletions.
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
using Backbone.BuildingBlocks.Application.Abstractions.Infrastructure.EventBus;
using Backbone.BuildingBlocks.Application.PushNotifications;
using Backbone.Modules.Devices.Application.Infrastructure.PushNotifications.Datawallet;
using Backbone.Modules.Devices.Domain.DomainEvents.Incoming.DatawalletModificationCreated;

namespace Backbone.Modules.Devices.Application.DomainEvents.Incoming.DatawalletModificationCreated;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
using Backbone.Modules.Devices.Application.DomainEvents.Incoming.DatawalletModificationCreated;
using Backbone.Modules.Devices.Application.DomainEvents.Incoming.ExternalEventCreated;
using Backbone.Modules.Devices.Application.DomainEvents.Incoming.IdentityDeletionProcessStarted;
using Backbone.Modules.Devices.Domain.DomainEvents.Incoming.DatawalletModificationCreated;
using Backbone.Modules.Devices.Domain.DomainEvents.Incoming.ExternalEventCreated;
using Backbone.Modules.Devices.Domain.DomainEvents.Outgoing;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
using Backbone.BuildingBlocks.Domain;
using Backbone.BuildingBlocks.Domain.Events;
using Backbone.DevelopmentKit.Identity.ValueObjects;

namespace Backbone.Modules.Devices.Application.DomainEvents.Incoming.DatawalletModificationCreated;
namespace Backbone.Modules.Devices.Domain.DomainEvents.Incoming.DatawalletModificationCreated;

public class DatawalletModifiedDomainEvent : DomainEvent
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ namespace Backbone.Modules.Devices.Domain.DomainEvents.Outgoing;

public class TierOfIdentityChangedDomainEvent : DomainEvent
{
public TierOfIdentityChangedDomainEvent(Identity identity, TierId oldTierIdId, TierId newTierIdId) : base($"{identity.Address}/TierOfIdentityChanged")
public TierOfIdentityChangedDomainEvent(Identity identity, TierId oldTierIdId, TierId newTierIdId) : base($"{identity.Address}/TierOfIdentityChanged/{Guid.NewGuid()}")
{
OldTierId = oldTierIdId;
NewTierId = newTierIdId;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ namespace Backbone.Modules.Synchronization.Domain.DomainEvents.Outgoing;

public class DatawalletModifiedDomainEvent : DomainEvent
{
public DatawalletModifiedDomainEvent(IdentityAddress identity, DeviceId modifiedByDevice) : base(Guid.NewGuid().ToString())
public DatawalletModifiedDomainEvent(IdentityAddress identity, DeviceId modifiedByDevice) : base($"{identity}/Datawallet/Modified/{Guid.NewGuid()}")
{
Identity = identity;
ModifiedByDevice = modifiedByDevice;
Expand Down
2 changes: 1 addition & 1 deletion helm/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -502,7 +502,7 @@ global:
# connectionInfo - only applicable if Vendor is "RabbitMQ"; the host name of the RabbitMQ service; the connection string of the Azure Service Bus if Vendor is "Azure"; if Vendor is "GoogleCloud" this is not used
connectionInfo: ""
# subscriptionClientName - the name of the subscription that should be used to receive events
subscriptionClientName: ""
subscriptionClientName: "consumerapi"
# rabbitMQUsername - only applicable if Vendor is "RabbitMQ"; should be set via environment variable
rabbitMQUsername: ""
# rabbitMQPassword - only applicable if Vendor is "RabbitMQ"; should be set via environment variable
Expand Down

0 comments on commit b40710e

Please sign in to comment.