Skip to content

Commit 43816ae

Browse files
authored
Merge pull request #121 from FlowTestAI/ui-improvements
feat: copy to clipboard and disclaimer for LLM usage
2 parents 2c3940f + bf4e714 commit 43816ae

16 files changed

Lines changed: 382 additions & 126 deletions

File tree

package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@
6161
"notistack": "^3.0.1",
6262
"postcss": "^8.4.35",
6363
"react": "^18.2.0",
64+
"react-copy-to-clipboard": "^5.1.0",
6465
"react-custom-scrollbars": "^4.2.1",
6566
"react-dom": "^18.2.0",
6667
"react-edit-text": "^5.1.1",

packages/flowtest-electron/src/ipc/settings.js

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,17 @@ const registerSettingsEventHandlers = (mainWindow) => {
2020
return Promise.reject(error);
2121
}
2222
});
23+
24+
ipcMain.handle('renderer:add-genAIUsageDisclaimer', async (event, accepted) => {
25+
try {
26+
settingsStore.addGenAIUsageDisclaimer(accepted);
27+
const savedSettings = settingsStore.getAll();
28+
29+
mainWindow.webContents.send('main:saved-settings', savedSettings);
30+
} catch (error) {
31+
return Promise.reject(error);
32+
}
33+
});
2334
};
2435

2536
module.exports = registerSettingsEventHandlers;

packages/flowtest-electron/src/store/settings.js

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,14 +9,20 @@ class Settings {
99
this.store.set('logSyncConfig', { enabled, hostUrl, accessId, accessKey });
1010
}
1111

12+
addGenAIUsageDisclaimer(accepted) {
13+
this.store.set('genAIUsageDisclaimer', accepted);
14+
}
15+
1216
getAll() {
1317
return {
1418
logSyncConfig: this.store.get('logSyncConfig') || {},
19+
genAIUsageDisclaimer: this.store.get('genAIUsageDisclaimer') || false,
1520
};
1621
}
1722

1823
clearAll() {
1924
this.store.set('logSyncConfig', {});
25+
this.store.set('genAIUsageDisclaimer', false);
2026
}
2127
}
2228

packages/flowtest-electron/tests/store/settings-store.test.js

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,21 @@ describe('settings-store', () => {
55
const store = new Settings();
66
store.clearAll();
77

8-
expect(store.getAll().logSyncConfig).toEqual({});
8+
let settings = store.getAll();
9+
expect(settings.logSyncConfig).toEqual({});
10+
expect(settings.genAIUsageDisclaimer).toEqual(false);
911

1012
// adding a collection whose directory doesn't exist
1113
store.addLogSyncConfig(true, 'http://localhost:3000', 'access_id', 'access_key');
12-
const config = store.getAll().logSyncConfig;
14+
store.addGenAIUsageDisclaimer(true);
15+
16+
settings = store.getAll();
17+
const config = settings.logSyncConfig;
1318
expect(config.enabled).toEqual(true);
1419
expect(config.hostUrl).toEqual('http://localhost:3000');
1520
expect(config.accessId).toEqual('access_id');
1621
expect(config.accessKey).toEqual('access_key');
22+
23+
expect(settings.genAIUsageDisclaimer).toEqual(true);
1724
});
1825
});

pnpm-lock.yaml

Lines changed: 48 additions & 36 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)