tbonelee commented on code in PR #5044:
URL: https://github.com/apache/zeppelin/pull/5044#discussion_r2304740997
##########
zeppelin-web-angular/src/app/core/paragraph-base/paragraph-base.ts:
##########
@@ -54,6 +55,7 @@ export abstract class ParagraphBase extends
MessageListenersManager {
// Initialized by `ViewChildren` in the class which extends ParagraphBase
notebookParagraphResultComponents!:
QueryList<NotebookParagraphResultComponent>;
+ notebookParagraphCodeEditorComponent!: NotebookParagraphCodeEditorComponent;
Review Comment:
I think this non-null assertion should be removed and changed to optional. I
found it to be `undefined` when `paragraph.config.editorHide === true` in the
`%md` editor case. Maybe you could check some errors after running `%md`
interpreter.
(`zeppelin-web-angular/src/app/pages/workspace/notebook/paragraph/paragraph.component.html`)
```html
<zeppelin-notebook-paragraph-code-editor
*ngIf="!paragraph.config.editorHide && !viewOnly"
...
```
I mistakenly added non-null assertion for
`notebookParagraphCodeEditorComponent` in `NotebookParagraphComponent` in a
previous commit, but that was incorrect. Sorry for the confusion.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscr...@zeppelin.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
