-
Notifications
You must be signed in to change notification settings - Fork 30
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
Webpack 5 upgrade #450
Webpack 5 upgrade #450
Conversation
Conflicting PR. Removed from build OMERO-plugins-push#1492. See the console output for more details.
--conflicts |
Conflicting PR. Removed from build OMERO-plugins-push#1493. See the console output for more details.
|
This fixes ECONNREFUSED errors when using localhost
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.
LGTM
README.rst
Outdated
|
||
**Notes**: | ||
You will usually want to have OMERO-iviewer installed on a local omero-web server, |
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.
OMERO.iviewer
README.rst
Outdated
|
||
**Notes**: | ||
You will usually want to have OMERO-iviewer installed on a local omero-web server, | ||
to test the JavaScript built above. NB: first uninstall iviewer if already installed, |
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.
OMERO.iviewer
README.rst
Outdated
|
||
The dev build of iviewer will attempt to connect to a local OMERO server at http://127.0.0.1:4080. |
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.
OMERO.iviewer
README.rst
Outdated
|
||
The initial data type (e.g. image, dataset, well) and its respective ID can be set/changed | ||
The iviewer will try to open an Image or other data from your local server, using IDs specified |
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.
OMERO.iviewer
Conflicting PR. Removed from build OMERO-plugins-push#157. See the console output for more details.
--conflicts |
Conflicting PR. Removed from build OMERO-plugins-push#158. See the console output for more details.
|
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.
Why is this included?
|
Removed |
Changes made as suggested. However, I'm not sure we really need to use |
Thanks |
Updated webpack, aurelia and lots of dependencies.
To test:
Various issues and tips from pages:
ome/omero-parade@115481e
https://webpack.js.org/migrate/5/
https://stackoverflow.com/questions/40379139/cannot-find-module-webpack-bin-config-yargs
aurelia/aurelia#982
aurelia/webpack-plugin#166
webpack/webpack-dev-server#1501
https://webpack.js.org/configuration/module/#ruleuse
https://webpack.js.org/configuration/dev-server/#devserverproxy
https://www.npmjs.com/package/expose-loader