Make company accounts info available to all authenticated users

ida-arbeitszeit / arbeitszeitapp

A webapp for labour-time calculation.

https://arbeitszeitapp.readthedocs.io

GNU Affero General Public License v3.0

33 stars 4 forks source link

Make company accounts info available to all authenticated users #1005

Closed seppeljordan closed 3 months ago

seppeljordan commented 3 months ago

This commit makes the account data for all companies available to all authenticated users. This was achieved by linking to the already implemented url routes in the company summary view.

Plan-ID: d21d8863-1179-4b7c-9d17-1701a7f78e76

sloschert commented 3 months ago

I think the navigation is currently a bit confusing, when you are viewing the account of a different company, because currently there is no information on the fact that it is not your own account. Also when you use the "tab" like navigation element that says "Accounts" you get navigated to the accounts overview of the foreign company, but again there is no hint that it's not your accounts.

I'd be ok with merging it anyway now, but we should remember to improve this.

seppeljordan commented 3 months ago

I think that we should overhaul (or reduce the complexity of) the navigation feature. Thanks for pointing that out.