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

設定のバックアップ作成時に名前を入力しなかった場合、ローカリゼーションがおかしくなる #13121

Closed
1 task done
zyoshoka opened this issue Feb 1, 2024 · 0 comments · Fixed by #13186
Labels
⚠️bug? This might be a bug packages/backend Server side specific issue/PR

Comments

@zyoshoka
Copy link
Contributor

zyoshoka commented Feb 1, 2024

💡 Summary

ページ /settings/preferences-backups の「新規作成」で設定のバックアップを作成する際、名前を入力せずに(空文字列ではなく null として)登録できますが、そのようにしてバックアップを作成した場合にローカライゼーションが正しく機能しなくなります。

🥰 Expected Behavior

  • name: null で作成してもローカライゼーションがおかしくならない

もしくは

  • name: null でバックアップを作成できない(空文字列になるようにするなど)

🤬 Actual Behavior

  • name: null で作成したバックアップを適用・削除する際にローカライゼーションが正しく機能しない
  • name: null でバックアップを作成できる

📝 Steps to Reproduce

No response

💻 Frontend Environment

* Model and OS of the device(s):
* Browser:
* Server URL:
* Misskey: 2024.2.0-beta.8

🛰 Backend Environment (for server admin)

No response

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
@zyoshoka zyoshoka added the ⚠️bug? This might be a bug label Feb 1, 2024
@zyoshoka zyoshoka changed the title 設定のバックアップ作成時に名前を入力せずに登録でき、そのように作成したバックアップを適用・削除することはできない 設定のバックアップ作成時に名前を入力しなかった場合、ローカリゼーションがおかしくなる Feb 1, 2024
@t1nyb0x t1nyb0x moved this to Triage in [BUG TRACKER] Backend Feb 5, 2024
@github-project-automation github-project-automation bot moved this from Triage to Done in [BUG TRACKER] Backend Feb 28, 2024
@samunohito samunohito added the packages/backend Server side specific issue/PR label Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug packages/backend Server side specific issue/PR
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants