Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Settings: Fix project overrides save #3708

Merged
merged 1 commit into from
Aug 23, 2022
Merged

Settings: Fix project overrides save #3708

merged 1 commit into from
Aug 23, 2022

Conversation

iLLiCiTiT
Copy link
Member

Brief description

Fix broken save of project overrides.

Description

There was wrong mongo operation when project overrides were saved to already existing overrides. This change is fixing it.

Testing notes:

  1. Save settings on a project
  2. Save the settings again and it should not crash

@iLLiCiTiT iLLiCiTiT self-assigned this Aug 23, 2022
@iLLiCiTiT iLLiCiTiT added the type: bug Something isn't working label Aug 23, 2022
@iLLiCiTiT iLLiCiTiT merged commit c74a6a8 into develop Aug 23, 2022
@iLLiCiTiT iLLiCiTiT deleted the bugfix/wrong_settings_update_operation branch August 23, 2022 08:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jakubjezek001 jakubjezek001 jakubjezek001 approved these changes

@64qam 64qam Awaiting requested review from 64qam

@m-u-r-p-h-y m-u-r-p-h-y Awaiting requested review from m-u-r-p-h-y

Assignees

@iLLiCiTiT iLLiCiTiT

Labels
type: bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@iLLiCiTiT @jakubjezek001