Change Details

This diff moves the logic for setting an avatar using your photo gallery from avatar hooks to edit thread avatar provider. Depends on D7679

In `selectFromGalleryAndUpdateThreadAvatar` we had a piece of state called `processingOrUploadInProgress` that would keep track of when we were uploading or processing the image the user selected in the setting image avatar process. We previously kept track of this state using a `boolean`; however, with this state now being used in this provider we need a more robust way to keep track of this state for a thread, and make sure this state is not shared across different threads. To fix this, I changed the type of this state to be a `Set<string>` from `boolean`. This set will keep track of all the thread ids that are actively uploading or processing an image for a new thread avatar. We then have `threadAvatarSaveInProgress` check if this set has the active `threadID` currently in this set will update the loader accordingly Depends on D7679

This diff moves the logic for setting an avatar using your photo gallery from avatar hooks to edit thread avatar provider.In `selectFromGalleryAndUpdateThreadAvatar` we had a piece of state called `processingOrUploadInProgress` that would keep track of when we were uploading or processing the image the user selected in the setting image avatar process. We previously kept track of this state using a `boolean`; however, with this state now being used in this provider we need a more robust way to keep track of this state for a thread, and make sure this state is not shared across different threads. To fix this, I changed the type of this state to be a `Set<string>` from `boolean`. This set will keep track of all the thread ids that are actively uploading or processing an image for a new thread avatar. We then have `threadAvatarSaveInProgress` check if this set has the active `threadID` currently in this set will update the loader accordingly Depends on D7679