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

Visual studio 2017 #700

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Visual studio 2017 #700

wants to merge 9 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Dec 11, 2017

I added a try/catch while iterating through providers so if fails (as it did with me) the application will not crash the application - it would not load.

I updated the Ribbon project for Visual Studio 2017 as I do not have Visual Studio 2015 on my system and it was complaining.

Billkrat added 2 commits December 11, 2017 12:11
TistoryBlogClient was not found causing application to crash.  Added try/catch and bypass errors so application can still execute
@dnfclas
Copy link

dnfclas commented Dec 11, 2017

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

❌ ghost sign now
You have signed the CLA already but the status is still pending? Let us recheck it.

@japj
Copy link

japj commented Dec 15, 2017

It seems appveyor does not have the windows 10 fall creator sdk (16299) yet, also looking at https://www.appveyor.com/docs/build-environment/#windows-sdks

Billkrat added 6 commits December 16, 2017 05:14
…ssing logic for plugins. Remarked out application updater since it was causing exceptions.
…ne failed to upload an image, caused the blog to be orphaned - couldn't do anything from BlogEngine and couldn't load in OpenWriter. Their is actually code in OpenWriter to resolve this but it couldn't be executed because of null reference error [resolved]
@martingbrown martingbrown mentioned this pull request Nov 5, 2018
@jongalloway
Copy link
Contributor

Is this a dupe of #776, or is there more in here to merge?

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