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.

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

Bug: Property values disappear when renaming property #742

Closed
chenilim opened this issue Jul 16, 2021 · 3 comments · Fixed by #797
Closed

Bug: Property values disappear when renaming property #742

chenilim opened this issue Jul 16, 2021 · 3 comments · Fixed by #797
Assignees
Labels
Bug Something isn't working Regression This is a regression from previously working functionality in a released version

Comments

@chenilim
Copy link
Member

Steps to reproduce the behavior:

  1. Open a card (e.g. from the Project management template)
  2. Rename a Select property (e.g. Priority)
  3. The values become "(UNKNOWN)"

Expected behavior:
The values should be unaffected by a property rename

Screenshots:
https://user-images.githubusercontent.com/46905241/125990822-37b94f62-1164-4503-a9eb-90bc2602d9c9.mov

Platform:

  • Focalboard plugin v0.8.1 (on community server)

Additional context:
This is a high-severity regression.

@chenilim chenilim added Bug Something isn't working Regression This is a regression from previously working functionality in a released version labels Jul 16, 2021
@chenilim
Copy link
Member Author

@hahmadia, could you help investigate this? It's a regression. We will want to cherry-pick a fix for release-0.8.0. Thanks. cc @harshilsharma63.

@kamre
Copy link
Contributor

kamre commented Jul 18, 2021

Regression was introduced in #553 when changing of type and name was merged into a single function Mutator.changePropertyTypeAndName because all options for new property template are always reset now and before there was a separate simple renaming.

Probably can be fixed with check if type was actually changed:

        if (propertyTemplate.type !== newType) {
            newTemplate.options = []
        }
        newTemplate.type = newType
        newTemplate.name = newName

@hahmadia hahmadia assigned harshilsharma63 and unassigned hahmadia Jul 22, 2021
@harshilsharma63
Copy link
Member

@kamre as always, thanks a ton for the investigation 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Regression This is a regression from previously working functionality in a released version
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants