[COMPASS-6060] Implement new Preferences class Created: 30/Aug/22  Updated: 28/Sep/22  Resolved: 14/Sep/22

Status: Closed
Project: Compass
Component/s: Preferences
Affects Version/s: None
Fix Version/s: 1.34.0

Type: Task Priority: Major - P3
Reporter: Anna Henningsen Assignee: Alena Khineika
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
is depended on by COMPASS-6070 Implement command-line argument parser Closed
is depended on by COMPASS-6071 Implement global config file parser Closed
is depended on by COMPASS-6092 Replace compass:usage:enabled/disable... Closed
is depended on by COMPASS-6107 Replace app:disable/enable-auto-updat... Closed
Related
is related to COMPASS-6131 Fix map rendering in schema analysis tab Closed
Epic Link: COMPASS-5816
Story Points: 5
Documentation Changes: Not Needed
Sprint: Iteration Jellyfish

 Description   

Implement the Preferences class as laid out in the technical design, to replace the Ampersand model as the public API of the compass-preferences-model package, and replace the Ampersand model’s usage throughout Compass.



 Comments   
Comment by Githook User [ 28/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: feat: implement new preferences class COMPASS-6060, COMPASS-6078 (#3435)

  • fix: update showedNetworkOptIn after merging from main
  • refactor: remove global config poc
  • feat: changes according to a new version of the tech design
  • feat: export UserPreferences type
  • feat: set default settings
  • refactor: remove extra check
  • build: add missing compass-preferences-model dependency
  • fix: resolve other depcheck issues
  • fix: return proper check
  • fix: modify getConfigurableUserPreferences
  • refactor: revert ipc as any
  • feat: create preferences ipc
  • docs: add comments to preferences model
  • refactor: use compass-utils
  • refactor: revert
  • refactor: clean up
  • fix: try different show tour logic
  • docs: add todo comment to tour closed
  • fix: do not listen for enableMaps and trackUsageStatistics
  • refactor: add some types
  • test: fix after refactroing
  • test: mock test-electron
  • test: fix for electron tests
  • test: fix intercom
  • refactor: onPreferencesChanged in model
  • feat: move preferences to typescript
  • test: remove only
  • docs: add type comment
  • refactor: fix some types
  • test: try to fix flaky test
Comment by Githook User [ 15/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: feat: implement new preferences class COMPASS-6060, COMPASS-6078 (#3435)

  • fix: update showedNetworkOptIn after merging from main
  • refactor: remove global config poc
  • feat: changes according to a new version of the tech design
  • feat: export UserPreferences type
  • feat: set default settings
  • refactor: remove extra check
  • build: add missing compass-preferences-model dependency
  • fix: resolve other depcheck issues
  • fix: return proper check
  • fix: modify getConfigurableUserPreferences
  • refactor: revert ipc as any
  • feat: create preferences ipc
  • docs: add comments to preferences model
  • refactor: use compass-utils
  • refactor: revert
  • refactor: clean up
  • fix: try different show tour logic
  • docs: add todo comment to tour closed
  • fix: do not listen for enableMaps and trackUsageStatistics
  • refactor: add some types
  • test: fix after refactroing
  • test: mock test-electron
  • test: fix for electron tests
  • test: fix intercom
  • refactor: onPreferencesChanged in model
  • feat: move preferences to typescript
  • test: remove only
  • docs: add type comment
  • refactor: fix some types
  • test: try to fix flaky test
Comment by Githook User [ 15/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: feat: implement new preferences class COMPASS-6060, COMPASS-6078 (#3435)

  • fix: update showedNetworkOptIn after merging from main
  • refactor: remove global config poc
  • feat: changes according to a new version of the tech design
  • feat: export UserPreferences type
  • feat: set default settings
  • refactor: remove extra check
  • build: add missing compass-preferences-model dependency
  • fix: resolve other depcheck issues
  • fix: return proper check
  • fix: modify getConfigurableUserPreferences
  • refactor: revert ipc as any
  • feat: create preferences ipc
  • docs: add comments to preferences model
  • refactor: use compass-utils
  • refactor: revert
  • refactor: clean up
  • fix: try different show tour logic
  • docs: add todo comment to tour closed
  • fix: do not listen for enableMaps and trackUsageStatistics
  • refactor: add some types
  • test: fix after refactroing
  • test: mock test-electron
  • test: fix for electron tests
  • test: fix intercom
  • refactor: onPreferencesChanged in model
  • feat: move preferences to typescript
  • test: remove only
  • docs: add type comment
  • refactor: fix some types
  • test: try to fix flaky test
Comment by Githook User [ 14/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: feat: implement new preferences class COMPASS-6060, COMPASS-6078 (#3435)

  • fix: update showedNetworkOptIn after merging from main
  • refactor: remove global config poc
  • feat: changes according to a new version of the tech design
  • feat: export UserPreferences type
  • feat: set default settings
  • refactor: remove extra check
  • build: add missing compass-preferences-model dependency
  • fix: resolve other depcheck issues
  • fix: return proper check
  • fix: modify getConfigurableUserPreferences
  • refactor: revert ipc as any
  • feat: create preferences ipc
  • docs: add comments to preferences model
  • refactor: use compass-utils
  • refactor: revert
  • refactor: clean up
  • fix: try different show tour logic
  • docs: add todo comment to tour closed
  • fix: do not listen for enableMaps and trackUsageStatistics
  • refactor: add some types
  • test: fix after refactroing
  • test: mock test-electron
  • test: fix for electron tests
  • test: fix intercom
  • refactor: onPreferencesChanged in model
  • feat: move preferences to typescript
  • test: remove only
  • docs: add type comment
  • refactor: fix some types
  • test: try to fix flaky test
Comment by Githook User [ 14/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: feat: implement new preferences class COMPASS-6060, COMPASS-6078 (#3435)

  • fix: update showedNetworkOptIn after merging from main
  • refactor: remove global config poc
  • feat: changes according to a new version of the tech design
  • feat: export UserPreferences type
  • feat: set default settings
  • refactor: remove extra check
  • build: add missing compass-preferences-model dependency
  • fix: resolve other depcheck issues
  • fix: return proper check
  • fix: modify getConfigurableUserPreferences
  • refactor: revert ipc as any
  • feat: create preferences ipc
  • docs: add comments to preferences model
  • refactor: use compass-utils
  • refactor: revert
  • refactor: clean up
  • fix: try different show tour logic
  • docs: add todo comment to tour closed
  • fix: do not listen for enableMaps and trackUsageStatistics
  • refactor: add some types
  • test: fix after refactroing
  • test: mock test-electron
  • test: fix for electron tests
  • test: fix intercom
  • refactor: onPreferencesChanged in model
  • feat: move preferences to typescript
  • test: remove only
  • docs: add type comment
  • refactor: fix some types
  • test: try to fix flaky test
Comment by Githook User [ 14/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: feat: implement new preferences class COMPASS-6060, COMPASS-6078 (#3435)

  • fix: update showedNetworkOptIn after merging from main
  • refactor: remove global config poc
  • feat: changes according to a new version of the tech design
  • feat: export UserPreferences type
  • feat: set default settings
  • refactor: remove extra check
  • build: add missing compass-preferences-model dependency
  • fix: resolve other depcheck issues
  • fix: return proper check
  • fix: modify getConfigurableUserPreferences
  • refactor: revert ipc as any
  • feat: create preferences ipc
  • docs: add comments to preferences model
  • refactor: use compass-utils
  • refactor: revert
  • refactor: clean up
  • fix: try different show tour logic
  • docs: add todo comment to tour closed
  • fix: do not listen for enableMaps and trackUsageStatistics
  • refactor: add some types
  • test: fix after refactroing
  • test: mock test-electron
  • test: fix for electron tests
  • test: fix intercom
  • refactor: onPreferencesChanged in model
  • feat: move preferences to typescript
  • test: remove only
  • docs: add type comment
  • refactor: fix some types
  • test: try to fix flaky test
Comment by Githook User [ 14/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: Merge remote-tracking branch 'origin/main' into COMPASS-6060-new-preferences-class

  1. Conflicts:
  2. packages/compass/src/main/application.ts
    Branch: preferences-types
    https://github.com/mongodb-js/compass/commit/8d6860347050bfdd0a82e7ccbd26c169402a9341
Comment by Githook User [ 14/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: Merge remote-tracking branch 'origin/main' into COMPASS-6060-new-preferences-class
Branch: preferences-types
https://github.com/mongodb-js/compass/commit/e55332e972842f202048cdb41248e30236415ccc

Comment by Githook User [ 14/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: Merge remote-tracking branch 'origin/main' into COMPASS-6060-new-preferences-class

  1. Conflicts:
  2. package-lock.json
  3. packages/compass-preferences-model/lib/model.js
  4. packages/compass-settings/src/utils/user-preferences.ts
  5. packages/compass-user-model/lib/model.js
    Branch: preferences-types
    https://github.com/mongodb-js/compass/commit/60839e717d1cb560357571adb3e564063e40b083
Comment by Githook User [ 14/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: feat: implement new preferences class COMPASS-6060, COMPASS-6078 (#3435)

  • fix: update showedNetworkOptIn after merging from main
  • refactor: remove global config poc
  • feat: changes according to a new version of the tech design
  • feat: export UserPreferences type
  • feat: set default settings
  • refactor: remove extra check
  • build: add missing compass-preferences-model dependency
  • fix: resolve other depcheck issues
  • fix: return proper check
  • fix: modify getConfigurableUserPreferences
  • refactor: revert ipc as any
  • feat: create preferences ipc
  • docs: add comments to preferences model
  • refactor: use compass-utils
  • refactor: revert
  • refactor: clean up
  • fix: try different show tour logic
  • docs: add todo comment to tour closed
  • fix: do not listen for enableMaps and trackUsageStatistics
  • refactor: add some types
  • test: fix after refactroing
  • test: mock test-electron
  • test: fix for electron tests
  • test: fix intercom
  • refactor: onPreferencesChanged in model
  • feat: move preferences to typescript
  • test: remove only
  • docs: add type comment
  • refactor: fix some types
  • test: try to fix flaky test
Comment by Githook User [ 13/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: Merge remote-tracking branch 'origin/main' into COMPASS-6060-new-preferences-class

  1. Conflicts:
  2. packages/compass/src/main/application.ts
    Branch: COMPASS-6060-new-preferences-class
    https://github.com/mongodb-js/compass/commit/8d6860347050bfdd0a82e7ccbd26c169402a9341
Comment by Githook User [ 13/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: Merge remote-tracking branch 'origin/main' into COMPASS-6060-new-preferences-class
Branch: COMPASS-6060-new-preferences-class
https://github.com/mongodb-js/compass/commit/e55332e972842f202048cdb41248e30236415ccc

Comment by Githook User [ 08/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: Merge remote-tracking branch 'origin/main' into COMPASS-6060-new-preferences-class

  1. Conflicts:
  2. package-lock.json
  3. packages/compass-preferences-model/lib/model.js
  4. packages/compass-settings/src/utils/user-preferences.ts
  5. packages/compass-user-model/lib/model.js
    Branch: COMPASS-6060-new-preferences-class
    https://github.com/mongodb-js/compass/commit/60839e717d1cb560357571adb3e564063e40b083
Comment by Githook User [ 08/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: feat: fetch preferences in main process COMPASS-6060
Branch: COMPASS-6060-new-preferences-class
https://github.com/mongodb-js/compass/commit/d82246e5e98326c6ff0c2b5b2a957b9809ed80c7

Comment by Githook User [ 08/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: feat: fetch preferences in main process COMPASS-6060
Branch: COMPASS-6060-new-preferences-class-ipc
https://github.com/mongodb-js/compass/commit/d82246e5e98326c6ff0c2b5b2a957b9809ed80c7

Comment by Githook User [ 06/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: Merge remote-tracking branch 'origin/main' into COMPASS-6060-new-preferences-class
Branch: COMPASS-6060-new-preferences-class
https://github.com/mongodb-js/compass/commit/543fcca89e6cab3626ea87998473cebc2e5003b3

Comment by Githook User [ 05/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: Merge remote-tracking branch 'origin/main' into COMPASS-6060-new-preferences-class

  1. Conflicts:
  2. packages/compass-auto-updates/electron/renderer/index.js
  3. packages/compass/src/app/index.js
  4. packages/compass/src/app/network-optin/index.js
    Branch: COMPASS-6060-new-preferences-class
    https://github.com/mongodb-js/compass/commit/f2ab7833edf437b7fb4eebea4b7bbea90ec4ee93
Comment by Githook User [ 05/Sep/22 ]

Author:

{'name': 'Alena Khineika', 'email': 'alena.khineika@gmail.com', 'username': 'alenakhineika'}

Message: feat: implement new preferences class COMPASS-6060
Branch: COMPASS-6060-new-preferences-class
https://github.com/mongodb-js/compass/commit/c099f9e37cebbf343e16c1ca6f3cc67717d2e965

Generated at Wed Feb 07 22:41:45 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.