Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Current user is assigned as owner on geoapp update #12261

Closed
giohappy opened this issue May 24, 2024 · 0 comments · Fixed by #12262
Closed

Current user is assigned as owner on geoapp update #12261

giohappy opened this issue May 24, 2024 · 0 comments · Fixed by #12262
Assignees
@mattiagiupponi
Milestone
4.3.0

Comments

@giohappy
Copy link
Contributor

Every time a geoapp is saved the current user is set as its owner.
The validate method inside the geoapps serializer assigns the current user as the resource (geoapp) owner. It's fine on creation but not on update!
@giohappy giohappy added this to the 4.3.0 milestone May 24, 2024
mattiagiupponi added a commit that referenced this issue May 24, 2024
@mattiagiupponi
[Fixes #12261] Current user is assigned as owner on geoapp update
eb0345d
@mattiagiupponi mattiagiupponi mentioned this issue May 24, 2024
Merged
12 tasks
@mattiagiupponi mattiagiupponi linked a pull request May 24, 2024 that will close this issue
[Fixes #12261] Current user is assigned as owner on geoapp update #12262
Merged
12 tasks
giohappy pushed a commit that referenced this issue May 24, 2024
@mattiagiupponi
[Fixes #12261] Current user is assigned as owner on geoapp update (#1…
012ca34 
…2262)
github-actions bot pushed a commit that referenced this issue May 24, 2024
@mattiagiupponi @github-actions
[Fixes #12261] Current user is assigned as owner on geoapp update (#1…
bcb16da 
…2262)

(cherry picked from commit 012ca34)
github-actions bot pushed a commit that referenced this issue May 24, 2024
@mattiagiupponi @github-actions
[Fixes #12261] Current user is assigned as owner on geoapp update (#1…
a199107 
…2262)

(cherry picked from commit 012ca34)
giohappy pushed a commit that referenced this issue May 24, 2024
@github-actions @mattiagiupponi
[Fixes #12261] Current user is assigned as owner on geoapp update (#1…
2ad8442 
…2262) (#12263)

(cherry picked from commit 012ca34)

Co-authored-by: mattiagiupponi <51856725+mattiagiupponi@users.noreply.github.com>
giohappy pushed a commit that referenced this issue May 24, 2024
@github-actions @mattiagiupponi
[Fixes #12261] Current user is assigned as owner on geoapp update (#1…
99e60ad 
…2262) (#12265)

(cherry picked from commit 012ca34)

Co-authored-by: mattiagiupponi <51856725+mattiagiupponi@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
@giohappy @mattiagiupponi