Merge pull request #2487 from pascalandr/fix/2405-settings-persistence
fix(settings): persist collapsed message preference
This commit is contained in:
@@ -565,6 +565,9 @@ export const createSettingsHelpers = (dependencies) => {
|
|||||||
result.userMessageRenderingMode = mode;
|
result.userMessageRenderingMode = mode;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
if (typeof candidate.collapsibleUserMessages === 'boolean') {
|
||||||
|
result.collapsibleUserMessages = candidate.collapsibleUserMessages;
|
||||||
|
}
|
||||||
if (typeof candidate.stickyUserHeader === 'boolean') {
|
if (typeof candidate.stickyUserHeader === 'boolean') {
|
||||||
result.stickyUserHeader = candidate.stickyUserHeader;
|
result.stickyUserHeader = candidate.stickyUserHeader;
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -74,6 +74,14 @@ describe('settings helpers', () => {
|
|||||||
expect(helpers.sanitizeSettingsUpdate({ wideChatLayoutEnabled: 'true' })).toEqual({});
|
expect(helpers.sanitizeSettingsUpdate({ wideChatLayoutEnabled: 'true' })).toEqual({});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('accepts only booleans for collapsible user messages', () => {
|
||||||
|
const helpers = createTestHelpers();
|
||||||
|
|
||||||
|
expect(helpers.sanitizeSettingsUpdate({ collapsibleUserMessages: true })).toEqual({ collapsibleUserMessages: true });
|
||||||
|
expect(helpers.sanitizeSettingsUpdate({ collapsibleUserMessages: false })).toEqual({ collapsibleUserMessages: false });
|
||||||
|
expect(helpers.sanitizeSettingsUpdate({ collapsibleUserMessages: 'true' })).toEqual({});
|
||||||
|
});
|
||||||
|
|
||||||
it('accepts messageStreamTransport as a persisted shared setting', () => {
|
it('accepts messageStreamTransport as a persisted shared setting', () => {
|
||||||
const helpers = createTestHelpers();
|
const helpers = createTestHelpers();
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user