opensearch-project / dashboards-notifications

The Notifications plugin provides a central location for all of your notifications from OpenSearch plugins. You can use either OpenSearch Dashboards or the REST API to configure notifications. Dashboards offers a more organized way of selecting a channel type and selecting which OpenSearch plugin sources you want to use.
Apache License 2.0
9 stars 36 forks source link

[BUG] Notifications dashboard doesn't install correctly in OSD #185

Closed derek-ho closed 4 months ago

derek-ho commented 5 months ago

Related PR: https://github.com/opensearch-project/dashboards-notifications/pull/165

Discovered via: https://github.com/opensearch-project/opensearch-dashboards-functional-test/pull/1146

ruanyl commented 4 months ago

I'm getting the same error, this seems related a recent commit https://github.com/opensearch-project/dashboards-notifications/commit/1fc2abe548017647b2e624ab9f4854e96e3d724d

ruanyl commented 4 months ago

cc @SuZhou-Joe @Hailong-am

SuZhou-Joe commented 4 months ago

Let me see.

SuZhou-Joe commented 4 months ago

@amsiglan Code from server directory should not rely on the code from public directory.

image

For the final build tar, the public directory of plugins will be bundled to target/{pluginId}.bundle.js, and server side's code won't be able to find the reference within public directory.

Hailong-am commented 4 months ago

this fixed by #189