Skip to content

Fix role schema to use camelCase naming convention#551

Draft
Copilot wants to merge 4 commits intoschemas/rolefrom
copilot/sub-pr-549-again
Draft

Fix role schema to use camelCase naming convention#551
Copilot wants to merge 4 commits intoschemas/rolefrom
copilot/sub-pr-549-again

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 13, 2026

The role schema used snake_case property names (role_name, created_at, updated_at, deleted_at) instead of the camelCase convention specified in the repository's README.

Changes

  • Schema properties: Updated role.yaml to use camelCase (roleName, createdAt, updatedAt, deletedAt)
  • Template: Updated role_template.json to match the new property names
  • Generated code: Regenerated models/v1beta1/role/role.go with corrected JSON/YAML struct tags
  • Imports: Restored pq and nulls package imports removed by code generator but required by other types in the file

Database column names remain snake_case via db tags; only JSON/YAML serialization uses camelCase.

// Before
RoleName string `db:"role_name" json:"role_name,omitempty" yaml:"role_name,omitempty"`
CreatedAt time.Time `db:"created_at" json:"created_at" yaml:"created_at"`

// After
RoleName string `db:"role_name" json:"roleName,omitempty" yaml:"roleName,omitempty"`
CreatedAt time.Time `db:"created_at" json:"createdAt" yaml:"createdAt"`

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 3 commits January 13, 2026 03:37
Co-authored-by: hortison <160366376+hortison@users.noreply.github.com>
Co-authored-by: hortison <160366376+hortison@users.noreply.github.com>
Co-authored-by: hortison <160366376+hortison@users.noreply.github.com>
Copilot AI changed the title [WIP] Add role schema definitions and helper functions Fix role schema to use camelCase naming convention Jan 13, 2026
Copilot AI requested a review from hortison January 13, 2026 03:40
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