Skip to content
This repository has been archived by the owner on Aug 13, 2023. It is now read-only.

Apply link to start-time #3130

Merged
merged 2 commits into from
Feb 13, 2020
Merged

Apply link to start-time #3130

merged 2 commits into from
Feb 13, 2020

Conversation

EinsteinNjoroge
Copy link
Contributor

@EinsteinNjoroge EinsteinNjoroge commented Feb 13, 2020 •
edited
Loading

Resolves #3063

Overall change: Apply the link in program-card to the start-time component.

Code changes:

  • Apply the link in program-card to the start-time component.
  • Bump package version.
  • I have assigned myself to this PR and the corresponding issues
  • Automated jest tests added (for new features) or updated (for existing features)
  • This PR requires manual testing

@EinsteinNjoroge EinsteinNjoroge self-assigned this Feb 13, 2020
@EinsteinNjoroge EinsteinNjoroge added radio-schedules ws-home Tasks for the WS Home Team labels Feb 13, 2020
@EinsteinNjoroge EinsteinNjoroge marked this pull request as ready for review February 13, 2020 14:05
AlistairGempf
AlistairGempf approved these changes Feb 13, 2020
View reviewed changes
Copy link
Contributor

@AlistairGempf AlistairGempf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

OlgaLyubin
OlgaLyubin approved these changes Feb 13, 2020
View reviewed changes
Copy link
Contributor

@OlgaLyubin OlgaLyubin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@EinsteinNjoroge EinsteinNjoroge merged commit f2126e9 into latest Feb 13, 2020
@sareh sareh deleted the apply-program-card-link-to-start-time-component branch February 21, 2020 12:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tochwill tochwill tochwill approved these changes

@OlgaLyubin OlgaLyubin OlgaLyubin approved these changes

@AlistairGempf AlistairGempf AlistairGempf approved these changes

Assignees

@EinsteinNjoroge EinsteinNjoroge

Labels
radio-schedules ws-home Tasks for the WS Home Team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Apply link to start time for Radio Schedules
4 participants
@EinsteinNjoroge @tochwill @OlgaLyubin @AlistairGempf