Skip to content

Conversation

@philippilievskibearingpointcom
Copy link
Collaborator

No description provided.

Comment on lines +6 to +8
const language = localStorage.getItem('language') || 'en';
console.log("SETTING LANGUAGE TO", language);
use(language);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is how plugins know which language to use?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exactly. It's pretty much how OpenSCD does it when loading plugins internally.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A lot of the files are not needed, you just copied in the whole thing, right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes i would need to refactor this and strip the imports i don't need 👍.

},
build: {
lib: {
entry: path.resolve(__dirname, 'apps/plugins/src/editors/Substation.ts'),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so this is not relevant as we build the plugins with the script right?

Copy link
Collaborator Author

@philippilievskibearingpointcom philippilievskibearingpointcom Apr 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exactly. This is just in the application for testing purposes right now. So i can quickly build a single plugin.
Will be removed when final version is ready.

@philippilievskibearingpointcom philippilievskibearingpointcom marked this pull request as ready for review April 22, 2025 11:42
@trusz trusz merged commit 43875d9 into com-pas:main May 2, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants