Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(iridium): add settings manager #3946

Merged
merged 23 commits into from
Jul 14, 2022
Merged

feat(iridium): add settings manager #3946

merged 23 commits into from
Jul 14, 2022

Conversation

jasonwoodland
Copy link
Contributor

@jasonwoodland jasonwoodland commented Jul 11, 2022

What this PR does πŸ“–

  • Implement iridium SettingsManager
  • Refactor IridiumManager so that the managers are not typed as optional (removes need for null checking at component-level)
  • Move the following settings to iridium:
    • Theme/flair
    • Flip video
    • Notification sounds
    • Keybinds
    • Privacy

Which issue(s) this PR fixes πŸ”¨
AP-1952

Special notes for reviewers πŸ—’οΈ

Additional comments 🎀

@jasonwoodland jasonwoodland added the draft A developer wants eyes on this PR, but they don't think it's ready to merge. label Jul 11, 2022
@github-actions github-actions bot added the Missing Dev Review A Dev and a Dev Approver need to review the PR, then mark as Ready for QA label Jul 11, 2022
@stavares843 stavares843 added iridium and removed Missing Dev Review A Dev and a Dev Approver need to review the PR, then mark as Ready for QA labels Jul 11, 2022
assets/styles/base.less Outdated Show resolved Hide resolved
assets/styles/base.less Outdated Show resolved Hide resolved
@jasonwoodland jasonwoodland removed the draft A developer wants eyes on this PR, but they don't think it's ready to merge. label Jul 13, 2022
@stavares843 stavares843 added the Missing Dev Review A Dev and a Dev Approver need to review the PR, then mark as Ready for QA label Jul 13, 2022
Copy link
Contributor

@josephmcg josephmcg left a comment

Choose a reason for hiding this comment

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

excellent work Jason, sets a strong foundation and creates some great utilities that can be implemented in other iridium modules

@molimauro molimauro removed the Missing Dev Review A Dev and a Dev Approver need to review the PR, then mark as Ready for QA label Jul 14, 2022
@josephmcg josephmcg merged commit 777f4ec into iridium-dev Jul 14, 2022
@josephmcg josephmcg deleted the AP1952 branch July 14, 2022 08:48
@github-actions github-actions bot removed the iridium label Jul 14, 2022
JustZacca pushed a commit that referenced this pull request Jul 14, 2022
* feat(iridium): add settings manager

* refactor(iridium): instantiate managers in constructor

* feat(iridium): add settings to iridium manager

* feat(settings): use iridium settings manager

* feat(theme): use iridium theme settings

* chore(store): remove old theme types/definitions

* feat(iridium): add init method for profile/group managers

* feat(theme): generate rgb color values for selection

* feat(settings): move flip local video setting to iridium

* chore(build): add deepmerge package

* feat(merge): add deepmerge utility function

* feat(settings): deep merge initial state

* feat(settings/audio): add notification sounds to settings

* feat(iridium): add keybinds to settings

* feat(settings): connect keybinds to iridium

* style(settings): use record for object types

* feat(iridium): add privacy to settings

* feat(settings): connect privacy to iridium

* refactor(settings): iterate over enum to generate initial state

* fix(build): bump @nuxtjs/eslint-config-typescript version

* style: organise imports

* fix(settings): only set in iridium if the value was changed

* fix(settings/audio): default values keys
JustZacca pushed a commit that referenced this pull request Jul 15, 2022
* feat(iridium): add settings manager

* refactor(iridium): instantiate managers in constructor

* feat(iridium): add settings to iridium manager

* feat(settings): use iridium settings manager

* feat(theme): use iridium theme settings

* chore(store): remove old theme types/definitions

* feat(iridium): add init method for profile/group managers

* feat(theme): generate rgb color values for selection

* feat(settings): move flip local video setting to iridium

* chore(build): add deepmerge package

* feat(merge): add deepmerge utility function

* feat(settings): deep merge initial state

* feat(settings/audio): add notification sounds to settings

* feat(iridium): add keybinds to settings

* feat(settings): connect keybinds to iridium

* style(settings): use record for object types

* feat(iridium): add privacy to settings

* feat(settings): connect privacy to iridium

* refactor(settings): iterate over enum to generate initial state

* fix(build): bump @nuxtjs/eslint-config-typescript version

* style: organise imports

* fix(settings): only set in iridium if the value was changed

* fix(settings/audio): default values keys
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.

None yet

4 participants