Skip to content

feat: Add parking as a GeofencingZoneCode - #404

Merged
marius-at-atb merged 1 commit into
mainfrom
marius/parkingGfz
Jun 24, 2026
Merged

feat: Add parking as a GeofencingZoneCode#404
marius-at-atb merged 1 commit into
mainfrom
marius/parkingGfz

Conversation

@marius-at-atb

@marius-at-atb marius-at-atb commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Resolves https://github.com/AtB-AS/kundevendt/issues/18890

Adds parking as gfz code.

@marius-at-atb
marius-at-atb requested a review from rosvik as a code owner June 24, 2026 08:52
@marius-at-atb
marius-at-atb requested review from Copilot and removed request for rosvik June 24, 2026 08:53
@marius-at-atb marius-at-atb self-assigned this Jun 24, 2026
@marius-at-atb
marius-at-atb requested review from jorunnl and rosvik June 24, 2026 08:53

Copilot AI left a comment

Copy link
Copy Markdown

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 updates the design-system theme typings to support a new geofencing zone code (parking) so downstream consumers can reference it as a valid GeofencingZoneCode.

Changes:

  • Add 'parking' to the GeofencingZoneCode union type.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +56 to +61
export type GeofencingZoneCode =
| 'allowed'
| 'slow'
| 'parking'
| 'noParking'
| 'noEntry';

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Waiting for update in Figma variables here, correct that the generated files also needs updates before we can use this in the app.

@marius-at-atb
marius-at-atb requested a review from Dahly96 June 24, 2026 12:11
@marius-at-atb
marius-at-atb merged commit bdd5ebe into main Jun 24, 2026
3 checks passed
@marius-at-atb
marius-at-atb deleted the marius/parkingGfz branch June 24, 2026 12:14
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.

3 participants