TeachLAFrontend
TeachLAFrontend copied to clipboard
Refactor: remove function type check
This commit introduced the following check:
https://github.com/uclaacm/TeachLAFrontend/blob/6f603af2500fedd7014795f1d9bfd38b9b005eed/src/components/Output/Output.js#L113-L118
This was meant to verify that getSrcDoc() is safe to run, but
- It should be
!(getSrcDoc && ... ); as written the condition is never true. - We supply
getSrcDocso we'd get an exception and find the problem ourselves ifgetSrcDocis not a function. The check isn't any more useful than an exception being thrown.