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

WindowsAzure.Storage #11

Open
StevenLangeraert opened this issue Mar 13, 2017 · 7 comments
Open

WindowsAzure.Storage #11

StevenLangeraert opened this issue Mar 13, 2017 · 7 comments

Comments

@StevenLangeraert
Copy link

StevenLangeraert commented Mar 13, 2017

Showing the log detail fails silently when using latest version (8.1.1) of the WindowsAzure.Storage nuget package,
due to removed signature Microsoft.WindowsAzure.Storage.Table.(Retrieve(String, String, List))

https://msdn.microsoft.com/library/azure/microsoft.windowsazure.storage.table.tableoperation.retrieve.aspx

@Jeavon
Copy link
Contributor

Jeavon commented Mar 13, 2017

@StevenLangeraert could you please the latest pre-release as I'm pretty sure we fixed this and upgraded the Storage dependency?

@StevenLangeraert
Copy link
Author

StevenLangeraert commented Mar 13, 2017

@Jeavon Thanks for the fast reply. Could it be there was a major breaking WindowsAzure.Storage nuget upgrade since then? Since I see comments on upgrading to WindowsAzure.Storage 7.2.1 but not to 8.0.0, neither can I find an implementation in source code that matches the Retrieve signatures of the latest (8.1.1) WindowsAzure.Storage nuget

@Jeavon
Copy link
Contributor

Jeavon commented Mar 13, 2017

It's possible although it seems the breaking change in WindowsAzure.Storage are undocumented, did you try the pre-release?
I'm happy to upgrade to v8.0.0 if needed...

@StevenLangeraert
Copy link
Author

With pre-release you mean

Install-Package Our.Umbraco.AzureLogger -Pre ?

or

Install-Package Our.Umbraco.AzureLogger -Version 0.2.1-alpha-000174

@Jeavon
Copy link
Contributor

Jeavon commented Mar 13, 2017

0.2.1-alpha-000174 from MyGet

@StevenLangeraert
Copy link
Author

StevenLangeraert commented Mar 13, 2017

Yes! That version is compatible with 8.1.1. not that I realy need that version, but I just like to use the latest ;-).

Sorry I overlooked that version on MyGet. Feel free to close the ticket ;-)

Thx for the cool plugin!

@Jeavon
Copy link
Contributor

Jeavon commented Mar 13, 2017

Great, think we will make a release of this today/tomorrow

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

No branches or pull requests

2 participants