-
Notifications
You must be signed in to change notification settings - Fork 72
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
お住まいの地域に空文字列が保存されないようにし、データベース内の空文字列や不正な値をnullへと変換するスクリプトを実行する #7628
Comments
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
お住まいの地域登録機能をReactからRailsのviewに移行しました。 このIssueを読む限りReactの時点では空文字列や不正な値が入っていたようです。 ReactからRailsのviewに移行後〜現在はデータベース内の空文字列や不正な値が入ってしまう問題が解決されたかは要調査が必要です。もし、現在でもその問題が残っている場合は、Issueにある通り、
この作業をお願いします。 現在の登録されているデータに関しても調査が必要ですが、Reactだった頃に入ってしまっていた空文字列や不正な値を修正したというコミットは見つからないので、空文字列や不正な値をnullへと変換するスクリプトを実行する必要はありそうです。 ...という訳で、本番データを見たりなどが必要なので、@SuzukiShuntarou さんは @komagata さんと連携を取りながら進めていただけたらと思います。 |
discordの質問時間に確認した内容をメモとして記載。 データベース本番環境におけるデータベースの更新
ビュー
|
概要
現在のアプリでは、ユーザーの登録情報変更においてお住まいの地域を登録しないを選んだ場合に、
country_code
とsubdivision_code
にはnullではなく空文字列が保存されるようになっています。これは望ましい挙動ではないため、もし選ばなかった場合にはnullが保存されるように変更したあと、データベースの空文字列や不正な値をnullへと変換するスクリプトを実行してデータベースから空文字列のデータを消去したいです。
その際にvalidationも現在の空文字列も許容出来る
allow_blank: true
から空文字列を許容しないallow_nil: true
へと変えてください。最終的に本番のデータベースでは
country_code
とsubdivision_code
にはnullかCountriesによって定義されているcountry codeとsubdivision codeの文字列のみが保存されていて、これらの値以外が保存されなくなればこのIssueは解決です。関連Issue
#6096
The text was updated successfully, but these errors were encountered: