Elsa/260511 fix description

[eltouma]

The description column can be NULL . During production deployment, scanning this nullable value into a Go string caused the error: name "description": converting NULL to string is unsupported

To handle nullable descriptions correctly, Description is now using sql.NullString instead of string.

This allows the code to handle descriptions that are either text or NULL.

doc: NullString

Summary by CodeRabbit

• Refactor

  • Improved description handling across the app to support nullable descriptions, ensuring consistent behavior for create, update, browse, and history views.

• Bug Fixes

  • Fixed rendering and validation of record descriptions in edit and display pages so empty/absent descriptions behave correctly.

[coderabbitai]

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Free

Run ID: ca425780-701d-484c-a83a-2e53cb63c90f

📥 Commits

Reviewing files that changed from the base of the PR and between 27732a7 and ab7e83c.

📒 Files selected for processing (1)

• src/record_handler.go

---

📝 Walkthrough

Walkthrough

The PR converts description fields from string to sql.NullString across the codebase. Models (Record, RecordHistory, PendingItem), handlers (CreateRecord, UpdateRecord, browse API), and templates are updated to handle nullable descriptions while preserving JSON tags.

Changes

Nullable Description Field Migration

Layer / File(s)	Summary
Data model struct definitions src/model.go, src/moderation.go	Record.Description, RecordHistory.Description, and PendingItem.Description are changed from string to sql.NullString; database/sql import is added.
Handler record creation, updating, and API response src/record_handler.go	CreateRecord and UpdateRecord validate description length with descriptionMaxLength and wrap submitted descriptions as sql.NullString (Valid set when non-empty); BrowseRecordShort and browse responses now use sql.NullString; supporting imports and minor formatting/validation renames updated.
Template display of nullable description src/templates/edit.html, src/templates/record.html	Templates access description values using the .String property of sql.NullString for display and editing.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 I nibble at strings and tuck them neat,
Into SQL's pockets, small and sweet,
Templates sip .String from a cozy bed,
Handlers pack Valid when words are said,
A rabbit's hop—safe nulls ahead!

---

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[NicolasCARPi]

👍