metabase / metabase/metabase

Add DB-level defaults for columns that deserve them

Open
#3,447 2 comments 0 reactions 0 assignees View on GitHub
:broom: .Backend .DX .Team/DevEx Difficulty:Easy Type:Tech Debt
Dominant language
Clojure
Stars
49.3k
Forks
6.8k
Avg merge
1d 13h
Merged PRs (30d)
653

Description

A prime example is the `:active` column we have for lots of models like Table, which has a `NOT NULL` constraint but no default value. A new table is obviously active when we insert it into the DB so we should make the default value of `:active` be `true`.

Contributor guide

Open the contributing guide

Research direction

Start by locating the Table model and the database schema or migration defining its :active column. Review other NOT NULL columns to determine which defaults belong at the database level, then verify that new inserts receive the intended defaults without weakening the constraints.

Written by the indexing model from the issue text.

Assessment

Tech stack
clojure
Domain
database
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.