Skip to content

fix the problems with project sharing#478

Open
josihoppe wants to merge 26 commits into
mainfrom
fix/sharing_rights
Open

fix the problems with project sharing#478
josihoppe wants to merge 26 commits into
mainfrom
fix/sharing_rights

Conversation

@josihoppe

Copy link
Copy Markdown
  • add decorator to use in views instead of repeating checks for user / viewer
  • check the problems described in:
  • adjust sharing rights for these problems

Comment thread app/dashboard/views.py Outdated
@josihoppe josihoppe marked this pull request as ready for review June 17, 2026 09:45
@paulapreuss

paulapreuss commented Jun 18, 2026

Copy link
Copy Markdown

@josihoppe Code looks nice, thanks for this refactoring! I think it will really improve maintenance regarding this topic ✨

I tested around a bit sharing with myself and I noticed the following:

  • If I have a project with read rights, I can still update the parameters on the assets in the energy system (asset_create_or_update). Once I do that, I am also able to restart the simulation. If there is no simulation on the scenario yet, I am also able to start a simulation. I only get a 403 if I try to reset the existing simulation with no parameters changed. That shouldn't be possibe just with read rights.
  • If a project is shared with me (both for edit and read rights), I do not see the button for project deletion, so I can't actually remove the project again from the projects view, although in the backend now it should work to just revoke the access.

After these things are fixed, I will rebase on main and hopefully have fixed the build dependencies and update the changelog before we merge.

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.

Graphs for the uploaded timeseries are not visible in a shared project

2 participants