Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 31, 2026

Addressed review feedback requesting verification that vehicle.maximumLevel is always a number and never null, confirming the design decision to hide level UI when schedule.level === null.

Verification Results

vehicle.maximumLevel is guaranteed non-null:

  • GraphQL schema: maximumLevel: Int! (non-nullable)
  • Database: maximum_charge smallint NOT NULL
  • TypeScript: maximumLevel: number
  • Vehicle creation always provides default values (e.g., Tesla defaults to 90)

schedule.level is intentionally nullable:

  • Used for schedule types that don't require level settings (e.g., Disable type)
  • GraphQL schema: level: Int (nullable)
  • Database: level smallint (nullable)

The v-if="localSchedule.level !== null" condition in commit f6057c5 correctly hides level UI only for schedules without level support.


💡 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 changed the title [WIP] Fix syncing schedule level display with local edits Verify vehicle.maximumLevel is always non-null as designed Jan 31, 2026
Copilot AI requested a review from fredli74 January 31, 2026 17:28
@fredli74 fredli74 closed this Jan 31, 2026
@fredli74 fredli74 deleted the copilot/sub-pr-946-again branch January 31, 2026 17:29
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