Archives/gristlabs_grist-core
1
0
Fork 0
mirror of https://github.com/gristlabs/grist-core.git synced 2024-10-27 20:44:07 +00:00
Code Issues Projects Releases Wiki Activity

(core) Remove deprecated code

Browse Source 
Summary:
The removed code is no longer reachable now that a new version of the plugin
API is being served by Grist on production.

Test Plan: Existing tests.

Reviewers: jarek

Reviewed By: jarek

Subscribers: jarek

Differential Revision: https://phab.getgrist.com/D4081
This commit is contained in:
George Gevoian 2023-10-17 23:25:33 -04:00
parent 91eabb6b0c
commit f6256646ef
3 changed files with 3 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
app/client/components/CustomView.ts
View File

@ -260,7 +260,6 @@ export class CustomView extends Disposable {
frame.useEvents(
ConfigNotifier.create(frame, this.viewSection, {
access,
theme: this.gristDoc.currentTheme,
}),
new MinimumLevel(AccessLevel.none)); // none access is enough
frame.useEvents(

10
app/client/components/WidgetFrame.ts
View File

@ -209,11 +209,7 @@ export class WidgetFrame extends DisposableWithEvents {
this.trigger('ready', this);
this._readyCalled.set(true);
}
if (
event.data.data?.message === 'themeInitialized' ||
// DEPRECATED: remove once the plugin API starts sending the message above.
event.data.data?.settings?.status === 'initialized'
) {
if (event.data.data?.message === 'themeInitialized') {
this._visible.set(true);
}
this._rpc.receiveMessage(event.data);
@ -572,7 +568,6 @@ export class RecordNotifier extends BaseEventSource {
export interface ConfigNotifierOptions {
access: AccessLevel;
theme: Computed<Theme>;
}
/**
@ -580,7 +575,6 @@ export interface ConfigNotifierOptions {
*/
export class ConfigNotifier extends BaseEventSource {
private _accessLevel = this._options.access;
private _theme = this._options.theme;
private _currentConfig = Computed.create(this, use => {
const options = use(this._section.activeCustomOptions);
return options;
@ -612,8 +606,6 @@ export class ConfigNotifier extends BaseEventSource {
options: this._currentConfig.get(),
settings: {
accessLevel: this._accessLevel,
// DEPRECATED: remove once the plugin API includes the `onThemeChange` handler.
...(fromReady ? {theme: this._theme.get()} : undefined),
},
fromReady,
});

2
app/plugin/grist-plugin-api.ts
View File

@ -247,6 +247,8 @@ async function getMappingsIfChanged(data: any): Promise<WidgetColumnMap|null> {
* Used by tests to wait for all pending requests to settle.
*
* TODO: currently only waits for requests for mappings.
*
* @internal
*/
export async function testWaitForPendingRequests() {
return await _activeRefreshReq;