-
Notifications
You must be signed in to change notification settings - Fork 304
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
Fix デフォルトプリセットの再登録が機能してないバグ(#1996)を修正 #2053
Fix デフォルトプリセットの再登録が機能してないバグ(#1996)を修正 #2053
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!!
実装ありがとうございます、とても良い実装だと思いました!!!
実際に動かしてみてプリセットがちゃんと消えてそうなことを確認しました!
ちょっとコメントの追加と関数の移動だけこちらでしてからマージさせていただこうと思います!!
あ、あとPRの名称も文脈わからなくてもわかるようにさせていただきます!
@madosuki ちょっと突発的なのですがご相談が! ニーズのある機能が実装されたときにSNSで言及しておりまして、今回のプルリクエストもツイートしたいと思っています。 (RTされたりリプライで届く感謝の言葉をお届けできればという意図と、あと開発者が分かるので新規コミッターの方がOSS開発に興味を持ってくださる導線を増やせればという意図があります・・・!) こんな感じを予定しています・・・!
(ちなみにアカウントあってますか・・・?) |
紹介問題ありません。 |
内容
マイグレーションのところのバージョン指定、とりあえず0.21にしてあります。動作確認は999.999.999にして行いました。
関連 Issue
close #1996
スクリーンショット・動画など
その他