openwisp / openwisp-notifications

Notifications module of OpenWISP
http://openwisp.org
GNU General Public License v3.0
39 stars 41 forks source link

chore: relative url for notification links #266

Closed Dhanus3133 closed 2 months ago

Dhanus3133 commented 4 months ago

Fixes: #249

image

Dhanus3133 commented 4 months ago

Updated with just the required notification widget changes, although the QA checks for the CSS files are failing for some reason.

Dhanus3133 commented 4 months ago

@nemesifier, I tried adding Selenium tests, but when the notification tab is clicked, it just loads. When I look into the console, it says it cannot find WS and returns 404 in the network. image Could you help me here? Thanks.

nemesifier commented 4 months ago

it just loads

Did you mean "it doesn't load"?

Dhanus3133 commented 4 months ago

Did you mean "it doesn't load"?

I meant the WebSocket connection is not found, so in the front end, it just waits, causing the spinning UI.

https://github.com/openwisp/openwisp-notifications/assets/43700516/1c21bb68-f2ec-4c4c-9fb5-da85fe4b112e