Skip to content

Conversation

@raboer
Copy link
Contributor

@raboer raboer commented Nov 18, 2025

goomens
goomens previously approved these changes Nov 18, 2025
…mes-versioning

# Conflicts:
#	UvA.Workflow/WorkflowInstances/IWorkflowInstanceRepository.cs
public DateTime Date { get; set; }
public DateTime? EventDate { get; set; }

[BsonElement("Operation")] public string Operation { get; set; } = null!;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not an enum?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea.. it is now!

…mes-versioning

# Conflicts:
#	UvA.Workflow.Api/Authentication/SurfConextAuthenticationHandler.cs
#	UvA.Workflow/Users/IUserService.cs
#	UvA.Workflow/Users/MockUserService.cs
#	UvA.Workflow/Users/UserService.cs
goomens
goomens previously approved these changes Nov 25, 2025
@goomens goomens dismissed their stale review November 25, 2025 03:15

The merge-base changed after approval.

@goomens goomens merged commit 4ec6d03 into main Nov 26, 2025
2 checks passed
@goomens goomens deleted the feature/DN-3423-Events-occurring-multiple-times-versioning branch November 26, 2025 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants