Reinstate Merge branch '388393-validate-attribute-presence' into 'master'
We reverted !125728 (merged) in this MR, this issue is to reinstate the MR with a proper fix
Related to #388393 (closed)
This is a follow-up of #423000 (closed) and is step 4 of making users and user_preferences columns NOT NULL.
Implementation plan
- Check that all the records have NOT NULL values using the below queries on a production clone:
Queries
SELECT COUNT(*)
FROM users
WHERE project_view IS NULL
OR hide_no_ssh_key IS NULL
OR hide_no_password IS NULL
OR notified_of_own_activity IS NULL;
SELECT COUNT(*)
FROM user_preferences
WHERE tab_width IS NULL
OR time_display_relative IS NULL
OR render_whitespace_in_code IS NULL;
- If there are records will NULL attribute then create a new issue to investigate what is setting those columns to NULL and then reschedule the background migrations introduced in !125881 (merged), !125774 (merged) to fix those records. Otherwise, continue with the below steps.
- Get this MR reviewed and merged !127587 (merged).
Next work on #423001 (closed).
Edited by Abdul Wadood