Differential D25502

[FlightPage] Fix display of airport
ClosedPublic
Actions

Authored by broulik on Nov 24 2019, 8:48 AM.

Details

Reviewers
vkrause
Commits
R1007:0d7d7510a9ba: [FlightPage] Fix display of airport
Summary

Handle the case when airport name isn't available

Test Plan

Now shows "TXL" instead of just "GERMANY"

Diff Detail

Repository
R1007 Itinerary - Travel Assistance Application
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Nov 24 2019, 8:48 AM
Restricted Application added a project: KDE PIM. · View Herald TranscriptNov 24 2019, 8:48 AM
Restricted Application added a subscriber: kde-pim. · View Herald Transcript
broulik requested review of this revision.Nov 24 2019, 8:48 AM
nicolasfella added a subscriber: nicolasfella.Nov 24 2019, 9:41 AM
nicolasfella added inline comments.
src/app/FlightPage.qml
39

else is not really needed since you return

vkrause accepted this revision.Nov 24 2019, 7:34 PM
This revision is now accepted and ready to land.Nov 24 2019, 7:34 PM
Closed by commit R1007:0d7d7510a9ba: [FlightPage] Fix display of airport (authored by broulik). · Explain WhyDec 5 2019, 8:04 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/app/FlightPage.qml (16 lines)

Diff 70935

View Options

src/app/FlightPage.qml

Loading...