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

finish rename of linkToCourse -> linkToAssigned. #15877

Merged
merged 1 commit into from
Jun 15, 2017

Conversation

Bjvanminnen
Copy link
Contributor

This change was meant to be part of the last commit in #15858, but apparently I'd failed to push it.

WIthout this, the server will be sending down a field with one name while the client expects something different. My dashboard tests already passed locally with this, so I'm going to push without waiting on circle.

@Bjvanminnen Bjvanminnen requested a review from Erin007 June 15, 2017 16:18
@Bjvanminnen Bjvanminnen merged commit e20d8b2 into staging Jun 15, 2017
@Bjvanminnen Bjvanminnen deleted the homepageSectionTable branch June 15, 2017 16:18
Copy link
Contributor

@ashercodeorg ashercodeorg left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@Erin007 Erin007 left a comment

Choose a reason for hiding this comment

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

👍🏼

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.

None yet

3 participants