-
Notifications
You must be signed in to change notification settings - Fork 290
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Export to html/pdf/python on Web when connected to remote server #10069
Conversation
@injectable() | ||
export class ExportToPDF implements INbConvertExport { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like all of your renames are not be detecting as renames? I usually force this with git add -A
but not sure if you can do it after the fact.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
didn't know I could do that, will try with next rename 💪
@inject(IExportDialog) protected readonly filePicker: IExportDialog, | ||
@inject(ExportUtil) protected readonly exportUtil: ExportUtil, | ||
@inject(INotebookImporter) protected readonly importer: INotebookImporter, | ||
@inject(ExportInterpreterFinder) private exportInterpreterFinder: ExportInterpreterFinder | ||
) {} | ||
|
||
public async export( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this method needed in these classes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still put it there since we register it as serviceManager.addSingleton<INbConvertExport>(INbConvertExport, ExportBase, 'Export Base');
, not sure why it's needed there. I can send a pr later to have it fixed.
import { ReportableAction } from '../progress/types'; | ||
|
||
@injectable() | ||
export class ExportBase implements INbConvertExport, IExportBase { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ExportHelper or Exporter is probably a better name now since they aren't being extended anymore since Base usually implies that it is a parent class
Codecov Report
@@ Coverage Diff @@
## main #10069 +/- ##
======================================
- Coverage 64% 64% -1%
======================================
Files 216 216
Lines 10060 10095 +35
Branches 1613 1618 +5
======================================
+ Hits 6508 6527 +19
- Misses 3028 3038 +10
- Partials 524 530 +6
|
Fixes #9996
package-lock.json
has been regenerated by runningnpm install
(if dependencies have changed).This PR covers following changes:
src/platform/progress/decorator.ts
andsrc/platform/progress/progressReporter.ts
are now common modules (nonode
api usage)exportToHTML
,exportToPython
,exportToPdf
,exportToPythonPlain
are common modulesexportBase
has two variations,exportBase.node
andexportBase.web
. Their major differences are:exportBase.web
doesn't create temp filesnbconvert
exportBase.node
runnbconvert
in local processexportBase.web
runnbconvert
through silent cell execution. It fetches the content throughcontentsManager
and save to workspace fs.fileConverter
as the common module contains most logics of how we export files, with some exceptions innode
env, which are overridden infileConverter.node