MobilityData / mobility-feed-api

Apache License 2.0
8 stars 3 forks source link

fix: 688 feed page minor UI fixes pre release #699

Closed qcdyx closed 1 month ago

qcdyx commented 1 month ago

Summary:

Closes #688

Expected behavior:

  1. Transit provider field in feed summary shouldn’t be bulleted when there’s only 1 provider

    image
  2. Authentication type shouldn't show if type is 0

    image
  3. Feed contact email shouldn't show a colon at the end

    image
  4. In cases where a dataset doesn't have a validation report, should show "Validation report not available” in the dataset history row

    image

Testing tips:

Provide tips, procedures and sample files on how to test the feature. Testers are invited to follow the tips AND to try anything they deem relevant outside the bounds of the testing tips.

Please make sure these boxes are checked before submitting your pull request - thanks!

github-actions[bot] commented 1 month ago

Preview Firebase Hosting URL: https://mobility-feeds-dev--pr-699-7543h9pe.web.app

emmambd commented 1 month ago

@qcdyx Can we remove the additional space at the end of the transit provider line?

qcdyx commented 1 month ago

@emmambd

image