Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Creating an Edge App #175

Closed
wants to merge 8 commits into from
Closed

Creating an Edge App #175

wants to merge 8 commits into from

Conversation

maudem
Copy link

@maudem maudem commented Jul 2, 2024

Story: https://phorge.wireload.net/T8616

Changes:

Update

  • Changed app to use Washington Metro Area Transit Authority's API because it had more data to work with
  • edge-apps/wmata-transit-status

To do:

  • Find list of delay types and set them to different colors
  • Create reademe and a list of stop to search through
  • Find a good way to display alert descriptions, they can be rather long

Screenshots:
Screenshot 2024-07-29 at 12 22 17 PM

@vpetersson
Copy link
Contributor

Please fix the location of this. It's overriding the existing TFL edge app.

@maudem
Copy link
Author

maudem commented Jul 3, 2024

Please fix the location of this. It's overriding the existing TFL edge app.
Is where it is in the file structure causing the problem?

@salmanfarisvp
Copy link
Collaborator

Hi @maudem, Sharing a few suggestions I have.

TFL App
image

The new APP
image

Could you please follow the TFL App design?

  • Station/Stop Name should be aligned to the and near to the Bus icon
  • Bus status should be listed near the ETA - When I run the app, I can't find the status option populated.
  • The App should support the standard screen size listed here.

Other General suggestions.

  • Use a specific name for PR
  • Use an application-specific name for the edge app - If it's a generic app, then it should named such as
  • Provide installation documentation, where to get the API and how to add..etc

@salmanfarisvp salmanfarisvp deleted the dev-onboarding branch October 11, 2024 00:23
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