Skip to content

Fix #8323 Add the name of the street driving on#8347

Open
alrega wants to merge 3 commits into
osmandapp:masterfrom
alrega:cur_street_name
Open

Fix #8323 Add the name of the street driving on#8347
alrega wants to merge 3 commits into
osmandapp:masterfrom
alrega:cur_street_name

Conversation

@alrega
Copy link
Copy Markdown

@alrega alrega commented Jan 28, 2020

I've modified source code in order to implement improvement #8323
I've tested changes by using Android Emulator, but not in the street using instant hardware.

Could you share APK file built in your CI with this change? I'd like to make system testing. I have aarch64 (arm64-v8a) CPU. Alternatively, please let me know Gradle task name to build APK locally.

Thank you in advance.

Fix bug in improper visibility evaluation of current street name
@alrega alrega requested review from Chumva and vshcherb February 1, 2020 16:14
@alrega
Copy link
Copy Markdown
Author

alrega commented Feb 7, 2020

@vshcherb could you review the changes and merge/rebase them?

Improve composing of current street name.
@alrega alrega requested a review from max-klaus February 9, 2020 08:59
@vshcherb vshcherb added this to the 3.8 milestone Feb 10, 2020
@vshcherb
Copy link
Copy Markdown
Member

We're not sure whether it fits all combinations or not, so we need more time to review it & test it.
There is no immediate plan to act on this request.

@vshcherb vshcherb modified the milestones: 3.8, Future milestone Mar 31, 2020
@vshcherb vshcherb added the Nice to Have Should be fixed but there is no priority or no possibility to fix it within current horizon planning label Dec 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Nice to Have Should be fixed but there is no priority or no possibility to fix it within current horizon planning

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants