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

Fix lack of support for unknown repositories in Upfile Editor window #32

Merged
merged 2 commits into from Dec 7, 2017

Conversation

niezbop
Copy link
Member

@niezbop niezbop commented Dec 5, 2017

The Edit Upfile window was failing because of the lack of support for the new GithubRepository. This fixes it by

  • adding support for GithubRepositories
  • handling unknown cases

@niezbop niezbop added the bug label Dec 5, 2017
@niezbop niezbop self-assigned this Dec 5, 2017
@niezbop niezbop requested a review from lacostej December 5, 2017 18:17
if(repository is GithubRepository)
{
GithubRepository temp = (GithubRepository) repository;
temp.Url = EditorGUILayout.TextField("Url of github repository", temp.Url);
Copy link
Member

Choose a reason for hiding this comment

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

do we need to support tags?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes we do! I missed that

@niezbop niezbop merged commit 552bbf5 into DragonBox:master Dec 7, 2017
@niezbop niezbop deleted the fix/upfile_editor branch December 7, 2017 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants