Skip to content

Conversation

irees
Copy link
Contributor

@irees irees commented Oct 7, 2025

No description provided.

@irees irees changed the title Port map components to typescript Port components to typescript Oct 8, 2025
@irees irees requested a review from Copilot October 14, 2025 23:27
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR ports Vue components to TypeScript by converting Vue 2 Options API components to Vue 3 Composition API with TypeScript type definitions.

  • Converts 50+ Vue components from Options API to Composition API with TypeScript
  • Adds comprehensive TypeScript interfaces for props, data, and API responses
  • Updates imports and exports to use TypeScript-compatible syntax

Reviewed Changes

Copilot reviewed 49 out of 50 changed files in this pull request and generated no comments.

File Description
src/runtime/plugins/filters.ts Moved makeRouteLink and makeStopLink functions to lib/filters.ts and imported them
src/runtime/lib/filters.ts Added exported functions makeRouteLink and makeStopLink with TypeScript signatures
src/runtime/components/*.vue Converted 50+ components from Options API to Composition API with TypeScript types
package.json Added @types/geojson dependency for GeoJSON type definitions
Comments suppressed due to low confidence (2)

src/runtime/components/stop-departures.vue:1

  • Added standard mask-image property alongside -webkit-mask-image for better cross-browser compatibility.
<template>

src/runtime/components/multi-service-levels.vue:1

  • Corrected spelling of 'number' from 'nunber' in comment.
<template>

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@irees irees merged commit e6cfee9 into main Oct 14, 2025
5 checks passed
@irees irees deleted the typescript-ports branch October 14, 2025 23:32
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.

1 participant