Make the new Dashboard the default page (#171)

Wojciech Kapcia (Tigase) opened 2 months ago

Currently, by default we show "index" page that lists all the available module.

In order to improve user experience new Dashboard should be the default view while the rest of the modules should be listed under "advanced menu" somewhere in the dashboard. To avoid problems with routings/paths Dashboard should still be served under /dashboard path but facilitate the default experience / path should redirect to /dashboard automatically

Activities

Andrzej Wójcik (Tigase) commented 4 days ago

I've implemented new redirect module and make it default on / with redirection to /dashboard. That should resolve this issue.

I had to slightly modify requestUri calculation (see RedirectServlet) as code for X-Forward-Prefix was not behaving correctly for redirects for /favicon.ico. Please review if my changes are correct.

IndexModule is now disabled, but I was wondering if we shouldn't just move it.
Andrzej Wójcik (Tigase) changed state to 'In QA' 4 days ago
Previous Value Current Value
Open

In QA
Andrzej Wójcik (Tigase) commented 4 days ago

As for adding advanced menu, I can work on that, but I wanted to confirm that current behavior is correct. Also, dashboard is not only for admins, so I wonder who should see this advanced menu, or if we should point/link to /index/ path with IndexModule being deployed there?
Wojciech Kapcia (Tigase) commented 3 days ago

I've implemented new redirect module and make it default on / with redirection to /dashboard. That should resolve this issue.

Works like a charm

I had to slightly modify requestUri calculation (see RedirectServlet) as code for X-Forward-Prefix was not behaving correctly for redirects for /favicon.ico. Please review if my changes are correct.

The substring? Seems fine.

IndexModule is now disabled, but I was wondering if we shouldn't just move it.

Hmm... could be useful.

As for adding advanced menu, I can work on that, but I wanted to confirm that current behavior is correct. Also, dashboard is not only for admins, so I wonder who should see this advanced menu, or if we should point/link to /index/ path with IndexModule being deployed there?

In general previous behaviour with index listing everything even wasn't too good as it could expose regular user to "admin stuff".

Maybe, for now, to limit the scope (Dashboard could be improved in the future) just add a link to /index/ (that would maybe only be visible to admin users) in the left hand menu (below "Server info")?
Wojciech Kapcia (Tigase) changed fields 3 days ago
Name Previous Value Current Value
Assignee

wojtek

andrzej.wojcik
Login to comment

Previous Value	Current Value
Open	In QA

Name	Previous Value	Current Value
Assignee	wojtek	andrzej.wojcik

Type	Improvement
Priority	Normal
Assignee	Andrzej Wójcik (Tigase)
Version	none
Sprints	n/a
Customer	n/a

Iterations

tigase-server-8.5.0 Open

Issue Votes (0)

Watchers (3)

Reference

tigase/_server/tigase-http-api#171