From 6dc83ff9bd6695db54bd558c155419480c9be066 Mon Sep 17 00:00:00 2001 From: Jonathan Bourdon Date: Wed, 4 Sep 2024 22:08:02 -0400 Subject: [PATCH] Ensure creator related blobs are stored in the creator container --- src/Application/AzureBlobStorage/Constants/ContainerNames.cs | 4 ++-- src/Web/Features/Contents/Handlers/ChangeBanner.cs | 3 +-- src/Web/Features/Contents/Handlers/ChangeLogo.cs | 2 +- src/Web/Features/Contents/Handlers/CreateContent.cs | 3 +-- src/Web/Features/Users/Handlers/ChangeAddress.cs | 1 - src/Web/Features/Users/Handlers/ChangeEmail.cs | 2 +- src/Web/Features/Users/Handlers/ChangePhone.cs | 2 +- 7 files changed, 7 insertions(+), 10 deletions(-) diff --git a/src/Application/AzureBlobStorage/Constants/ContainerNames.cs b/src/Application/AzureBlobStorage/Constants/ContainerNames.cs index 1f49c7e..039a27d 100644 --- a/src/Application/AzureBlobStorage/Constants/ContainerNames.cs +++ b/src/Application/AzureBlobStorage/Constants/ContainerNames.cs @@ -2,6 +2,6 @@ public static class ContainerNames { - public static string Users = "users"; - public static string Creators = "creators"; + public const string Users = "users"; + public const string Creators = "creators"; } diff --git a/src/Web/Features/Contents/Handlers/ChangeBanner.cs b/src/Web/Features/Contents/Handlers/ChangeBanner.cs index d33e72a..c13aa95 100644 --- a/src/Web/Features/Contents/Handlers/ChangeBanner.cs +++ b/src/Web/Features/Contents/Handlers/ChangeBanner.cs @@ -43,9 +43,8 @@ public class ChangeBannerHandler( return; } - // TODO: this upload should be done to the Creators container var blobUrl = await blobStorage.UploadFileAsync( - ContainerNames.Users, + ContainerNames.Creators, $"{request.CreatorId}/{SubDirectoryNames.Profile}/{CommonFileNames.BannerPicture}", request.File.OpenReadStream(), request.File.ContentType, diff --git a/src/Web/Features/Contents/Handlers/ChangeLogo.cs b/src/Web/Features/Contents/Handlers/ChangeLogo.cs index 656bf62..0caf367 100644 --- a/src/Web/Features/Contents/Handlers/ChangeLogo.cs +++ b/src/Web/Features/Contents/Handlers/ChangeLogo.cs @@ -56,7 +56,7 @@ public class ChangeLogoHandler( // TODO: this upload should be done to the Creators container var blobUrl = await blobStorage.UploadFileAsync( - ContainerNames.Users, + ContainerNames.Creators, $"{request.CreatorId}/{SubDirectoryNames.Profile}/{CommonFileNames.ProfilePicture}", request.File.OpenReadStream(), request.File.ContentType, diff --git a/src/Web/Features/Contents/Handlers/CreateContent.cs b/src/Web/Features/Contents/Handlers/CreateContent.cs index cd8934a..3a0dd6f 100644 --- a/src/Web/Features/Contents/Handlers/CreateContent.cs +++ b/src/Web/Features/Contents/Handlers/CreateContent.cs @@ -138,9 +138,8 @@ public sealed class PostContent( IFormFile file, CancellationToken ct = default) { - // TODO: I would like us to use ContainerNames.Creators but it seems we are missing configurations @jbourdon var url = await blobStorage.UploadFileAsync( - ContainerNames.Users, + ContainerNames.Creators, $"{creatorId}/{SubDirectoryNames.Contents}/{contentId}/{file.FileName}", file.OpenReadStream(), file.ContentType, diff --git a/src/Web/Features/Users/Handlers/ChangeAddress.cs b/src/Web/Features/Users/Handlers/ChangeAddress.cs index b139f44..7eea146 100644 --- a/src/Web/Features/Users/Handlers/ChangeAddress.cs +++ b/src/Web/Features/Users/Handlers/ChangeAddress.cs @@ -31,7 +31,6 @@ public class ChangeAddressHandler( } user.Address = request.Address; - // TODO: check to see if identity resets the email-validated flag - @jonathan var result = await userManager.UpdateAsync(user); diff --git a/src/Web/Features/Users/Handlers/ChangeEmail.cs b/src/Web/Features/Users/Handlers/ChangeEmail.cs index 8f7c844..9af2864 100644 --- a/src/Web/Features/Users/Handlers/ChangeEmail.cs +++ b/src/Web/Features/Users/Handlers/ChangeEmail.cs @@ -31,8 +31,8 @@ public class ChangeEmailHandler( } user.Email = request.Email; - // TODO: check to see if identity resets the email-validated flag - @jonathan + // TODO: check to see if identity resets the `email confirmed` flag - @jonathan var result = await userManager.UpdateAsync(user); if (result.Succeeded) diff --git a/src/Web/Features/Users/Handlers/ChangePhone.cs b/src/Web/Features/Users/Handlers/ChangePhone.cs index 8e7a91f..4341ff2 100644 --- a/src/Web/Features/Users/Handlers/ChangePhone.cs +++ b/src/Web/Features/Users/Handlers/ChangePhone.cs @@ -31,7 +31,7 @@ public class ChangePhoneHandler( } user.PhoneNumber = request.PhoneNumber; - // TODO: check to see if identity resets the email-validated flag - @jonathan + // TODO: check to see if identity resets the `phone confirmed` flag - @jonathan var result = await userManager.UpdateAsync(user);