Skip to content

Conversation

@EvenModiguy
Copy link
Contributor

No description provided.

RachikAbidi and others added 30 commits December 9, 2024 14:02
Co-authored-by: Mike PETITFILS <mickael.petitfils@hove.com>
@EvenModiguy EvenModiguy added the REVIEW REQUIRED 👀 A review is required label Mar 14, 2025
@EvenModiguy EvenModiguy requested a review from RachikAbidi March 25, 2025 15:34
try AroundMe.shared.initialize(
coverage: "fr-idf",
token: "your_token",
timeZone: "your_contry",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
timeZone: "your_contry",
timeZone: "your_country",

<h2>🗓 24 Feb 2025</h2>

#### Feature
- Show/hide PMR availability based on configuration
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Show/hide PMR availability based on configuration
- Show/hide PRM availability based on configuration

- Show/hide PMR availability based on configuration

#### Tasks
- Change configuration
Copy link
Contributor

Choose a reason for hiding this comment

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

This changelog needs more details.

Suggested change
- Change configuration
- Extract module configuration from unified configuration
- Add unified colors configuration

<h2>🗓 24 Mar 2025</h2>

#### Tasks
- Handle favorite journeys with park mode sections to hide it on the freeze
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Handle favorite journeys with park mode sections to hide it on the freeze
- Handle favorite journeys with park mode sections


#### Tasks
- Handle favorite journeys with park mode sections to hide it on the freeze
- Handle autocompletion behavior to be the same than the Android version
Copy link
Contributor

Choose a reason for hiding this comment

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

This is unacceptable, mentioning Android in an iOS release note is completely useless. Please reformulate this change, specify what has changed exactly

- Handle favorite journeys with park mode sections to hide it on the freeze
- Handle autocompletion behavior to be the same than the Android version
- Add config params to override favorite tabs title
- Display POI icons if available
Copy link
Contributor

Choose a reason for hiding this comment

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

Where the icon is displayed?

- Handle autocompletion behavior to be the same than the Android version
- Add config params to override favorite tabs title
- Display POI icons if available
- Display line details for every level of zoom
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Display line details for every level of zoom
- Display line details for every zoom level


#### Fixes
- Fix transport line text color icon in map annotation
- Fix back button action: previous state is shown when show back
Copy link
Contributor

Choose a reason for hiding this comment

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

Can't understand this change

#### Fixes
- Fix transport line text color icon in map annotation
- Fix back button action: previous state is shown when show back
- Fix geojson's mapping to build polylines
Copy link
Contributor

Choose a reason for hiding this comment

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

Needs to be more explicit

@RachikAbidi RachikAbidi merged commit fff74c6 into master Mar 28, 2025
1 check passed
@RachikAbidi RachikAbidi deleted the release/13-02-25 branch March 28, 2025 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

REVIEW REQUIRED 👀 A review is required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants