Skip to content

Fix UUID PR review issues: aliases, dashboard_assigns, and naming con…#337

Merged
ddon merged 1 commit intoBeamLabEU:devfrom
mdon:dev
Feb 14, 2026
Merged

Fix UUID PR review issues: aliases, dashboard_assigns, and naming con…#337
ddon merged 1 commit intoBeamLabEU:devfrom
mdon:dev

Conversation

@mdon
Copy link

@mdon mdon commented Feb 14, 2026

…sistency

  • Move 16 alias-in-function-body instances to module level across publishing, shop, emails, and update task modules
  • Apply dashboard_assigns() to 13 LiveViews passing raw assigns to Layouts.dashboard (prevents redundant layout diffs)
  • Rename *_id assigns to *_uuid in billing forms for convention consistency (order_form, subscription_form, billing_profile_form)
  • Add UUID naming convention report documenting 25 Pattern 2 schemas and connections module FK analysis

…sistency

- Move 16 alias-in-function-body instances to module level across
  publishing, shop, emails, and update task modules
- Apply dashboard_assigns() to 13 LiveViews passing raw assigns to
  Layouts.dashboard (prevents redundant layout diffs)
- Rename *_id assigns to *_uuid in billing forms for convention
  consistency (order_form, subscription_form, billing_profile_form)
- Add UUID naming convention report documenting 25 Pattern 2 schemas
  and connections module FK analysis

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ddon ddon merged commit 86e97a4 into BeamLabEU:dev Feb 14, 2026
6 checks passed
ddon pushed a commit that referenced this pull request Feb 14, 2026
- Mark connections module type: :integer as correct (not a bug)
- Track billing form assign renames, dashboard_assigns, and alias fixes
- Add Pattern 2 naming decision as pending item
- Reorganize priority lists and checklists to reflect current state

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants