diff --git a/src/sql/workbench/contrib/notebook/browser/models/notebookInput.ts b/src/sql/workbench/contrib/notebook/browser/models/notebookInput.ts
index a6703784ce..c95b086353 100644
--- a/src/sql/workbench/contrib/notebook/browser/models/notebookInput.ts
+++ b/src/sql/workbench/contrib/notebook/browser/models/notebookInput.ts
@@ -247,6 +247,10 @@ export abstract class NotebookInput extends EditorInput {
 		return this._title;
 	}
 
+	public isReadonly(): boolean {
+		return false;
+	}
+
 	public async getProviderInfo(): Promise<IProviderInfo> {
 		await this._providersLoaded;
 		return {
diff --git a/src/sql/workbench/contrib/query/common/queryEditorInput.ts b/src/sql/workbench/contrib/query/common/queryEditorInput.ts
index fe79446acc..fff8838fde 100644
--- a/src/sql/workbench/contrib/query/common/queryEditorInput.ts
+++ b/src/sql/workbench/contrib/query/common/queryEditorInput.ts
@@ -177,6 +177,10 @@ export abstract class QueryEditorInput extends EditorInput implements IConnectab
 	public supportsSplitEditor(): boolean { return false; }
 	public revert(): Promise<boolean> { return this._text.revert(); }
 
+	public isReadonly(): boolean {
+		return false;
+	}
+
 	public matches(otherInput: any): boolean {
 		// we want to be able to match against our underlying input as well, bascially we are our underlying input
 		if (otherInput instanceof QueryEditorInput) {