Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Fixed redundant and confusing location settings. #7158

Merged
merged 1 commit into from
Nov 1, 2022

Conversation

barankyle
Copy link
Member

@barankyle barankyle commented Oct 31, 2022

Summary

location_settings had both audioEnabled and instanceMediaChatEnabled. The latter was what controlled whether audio was on, while the former didn't do anything. Removed instanceMediaChatEnabled and replaced its usage with audioEnabled.

References

closes #insert number here

Checklist

  • If this PR is still a WIP, convert to a draft
  • ensure all checks pass
  • When this PR is ready, mark it as "Ready for review"
  • Changes have been manually QA'd
  • Changes reviewed by at least 2 approved reviewer

QA Steps

List any additional steps required to QA the changes of this PR, as well as any supplemental images or videos.

location_settings had both audioEnabled and instanceMediaChatEnabled. The latter was what
controlled whether audio was on, while the former didn't do anything. Removed instanceMediaChatEnabled
and replaced its usage with audioEnabled.
@HexaField HexaField merged commit db10818 into dev Nov 1, 2022
@HexaField HexaField deleted the cleanup-location-settings branch November 1, 2022 07:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants