Skip to content
This repository was archived by the owner on Dec 5, 2024. It is now read-only.

Calling UpdateConfigData in BranchView.OnEnable() to refresh BranchList #431

Merged
StanleyGoldman merged 2 commits into from
Nov 13, 2017
Merged

Calling UpdateConfigData in BranchView.OnEnable() to refresh BranchList #431

StanleyGoldman merged 2 commits into from
Nov 13, 2017

Conversation

@StanleyGoldman
Copy link
Contributor

@StanleyGoldman StanleyGoldman commented Nov 13, 2017 •
edited
Loading

Update to #428
Removing the functionality to call UpdateConfigData after commit and calling it through BranchView.OnEnable() instead.

@StanleyGoldman StanleyGoldman requested a review from shana November 13, 2017 16:11
shana
shana approved these changes Nov 13, 2017
View reviewed changes
Copy link
Member

@shana shana left a comment

Choose a reason for hiding this comment

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

👍

@StanleyGoldman StanleyGoldman merged commit 64e45bd into master Nov 13, 2017
@StanleyGoldman StanleyGoldman deleted the fixes/refresh-status-and-log-patch branch November 13, 2017 17:40
StanleyGoldman added a commit that referenced this pull request Nov 13, 2017
@StanleyGoldman
Revert "Merge pull request #431 from github-for-unity/fixes/refresh-s…
022b266 
…tatus-and-log-patch"

This reverts commit 64e45bd, reversing
changes made to ad226dd.
@StanleyGoldman StanleyGoldman mentioned this pull request Nov 13, 2017
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@shana shana shana approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

ready-for-review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@StanleyGoldman @shana