mirror of
https://github.com/jupyter/notebook.git
synced 2025-03-01 12:56:54 +08:00
Merge pull request #262 from jtpio/fix-plotly
Fix handling of federated mime extensions
This commit is contained in:
commit
2990526ecb
23
app/index.js
23
app/index.js
@ -69,15 +69,13 @@ async function main() {
|
|||||||
// load extra packages
|
// load extra packages
|
||||||
require('@jupyterlab/celltags');
|
require('@jupyterlab/celltags');
|
||||||
|
|
||||||
const mimeExtensions = [
|
const mimeExtensionsMods = [
|
||||||
require('@jupyterlab/javascript-extension'),
|
require('@jupyterlab/javascript-extension'),
|
||||||
require('@jupyterlab/json-extension'),
|
require('@jupyterlab/json-extension'),
|
||||||
require('@jupyterlab/pdf-extension'),
|
require('@jupyterlab/pdf-extension'),
|
||||||
require('@jupyterlab/vega5-extension')
|
require('@jupyterlab/vega5-extension')
|
||||||
];
|
];
|
||||||
|
const mimeExtensions = await Promise.all(mimeExtensionsMods);
|
||||||
const RetroApp = require('@retrolab/application').RetroApp;
|
|
||||||
const app = new RetroApp({ mimeExtensions });
|
|
||||||
|
|
||||||
const disabled = [];
|
const disabled = [];
|
||||||
// TODO: formalize the way the set of initial extensions and plugins are specified
|
// TODO: formalize the way the set of initial extensions and plugins are specified
|
||||||
@ -302,6 +300,20 @@ async function main() {
|
|||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
|
// Add the federated mime extensions.
|
||||||
|
const federatedMimeExtensions = await Promise.allSettled(
|
||||||
|
federatedMimeExtensionPromises
|
||||||
|
);
|
||||||
|
federatedMimeExtensions.forEach(p => {
|
||||||
|
if (p.status === 'fulfilled') {
|
||||||
|
for (let plugin of activePlugins(p.value)) {
|
||||||
|
mimeExtensions.push(plugin);
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
console.error(p.reason);
|
||||||
|
}
|
||||||
|
});
|
||||||
|
|
||||||
// Load all federated component styles and log errors for any that do not
|
// Load all federated component styles and log errors for any that do not
|
||||||
(await Promise.allSettled(federatedStylePromises))
|
(await Promise.allSettled(federatedStylePromises))
|
||||||
.filter(({ status }) => status === 'rejected')
|
.filter(({ status }) => status === 'rejected')
|
||||||
@ -309,6 +321,9 @@ async function main() {
|
|||||||
console.error(reason);
|
console.error(reason);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
const RetroApp = require('@retrolab/application').RetroApp;
|
||||||
|
const app = new RetroApp({ mimeExtensions });
|
||||||
|
|
||||||
app.registerPluginModules(mods);
|
app.registerPluginModules(mods);
|
||||||
|
|
||||||
// Expose global app instance when in dev mode or when toggled explicitly.
|
// Expose global app instance when in dev mode or when toggled explicitly.
|
||||||
|
Loading…
Reference in New Issue
Block a user