Skip to content

Display delay minutes in parentheses - #47

Open
michael-fritzsch wants to merge 1 commit into
nilstgmd:mainfrom
michael-fritzsch:display-delay
Open

Display delay minutes in parentheses#47
michael-fritzsch wants to merge 1 commit into
nilstgmd:mainfrom
michael-fritzsch:display-delay

Conversation

@michael-fritzsch

Copy link
Copy Markdown
Contributor

Summary

The API's departure time already includes delays. I think it is clearer if the delay is shown in parentheses to account for this.

Example: Train's planned departure is 12:03 with current delay of 2 minutes. This would currently be shown as 12:05 +2 min. I propose to change this to 12:05 (+2 min).

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • CI/CD or build changes

Changes Made

  • Add parenthesis around delay time

Checklist

  • I have tested my changes locally
  • I have run npm test and all tests pass
  • I have added tests for new functionality (if applicable)
  • I have updated the README.md (if applicable)
  • My code follows the project's code style
  • I have added comments to my code where necessary

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.

1 participant