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

Project status 2022-01? #36

Open
chrisgull opened this issue Jan 17, 2022 · 7 comments
Open

Project status 2022-01? #36

chrisgull opened this issue Jan 17, 2022 · 7 comments

Comments

@chrisgull
Copy link

Hi,
I am wondering if this project is still alive?
Thanks,
Lars

@AlexGhiondea
Copy link
Owner

Hey

It's still 'alive' but I haven't spent too much time on active development. Is there an area you would like to see more stuff being done?

@chrisgull
Copy link
Author

chrisgull commented Jan 17, 2022

Alex, I have been using this package for Smugmug image downloads, am now looking at uploads to Smugmug. With migration to .NET 6 I am of course concerned with depending on a package that would not be maintained. The fact that there is no sharp release (just beta) worried me as well. Perhaps you can comment on this. I think for my use case I am looking for .NET Standard 2.0 packaging. I think that's what you have?

BTW since there is only prerelease with Nuget on this project, it does not show up on the package search in VS unless you check include prereleases. Makes it difficult to find with a casual search inside VS. Maybe you want a wider audience?

BTW2 the link from your README to codeplex.com is dead.
Best,
Lars

@AlexGhiondea
Copy link
Owner

There are a couple of things I would need to do before I can drop the preview -- there are some APIs that are not yet wrapped the way I want it.

I think the package supports .NET Standard 2.0.

I will see if I can do some work on this project in the weekend -- if you are interested, I would be open to taking contributions as well!

@chrisgull
Copy link
Author

Not sure if I have the knowledge at this point to contribute much. There are two areas that come to mind:

  • Sample code: I have been using your test code as sample code, but it's pretty limited. For example as far as I can tell uploads are not covered.
  • SmugMugAuthorize.AuthorizeSmugMug() is console app only. For my use case, with async UI functions like a message box prompting for a reply, I forked it into my own async function to support my UI flow. I'm not sure if the package should provide this function, as UI models differ quite a bit. And I think it should be async either way, it seems like a good pattern to always let the caller handle await.

@AlexGhiondea
Copy link
Owner

Samples is actually a great place to start in my experience!

As for the auth flow -- oAuth is so tied to the web that anything that is not a web app will have to make some adjustments/tweaks.

@chrisgull
Copy link
Author

chrisgull commented Jan 18, 2022

Alex,
I have been using the Smugmug.Net.v2 namespace, am now realizing that I should interface with the SmugMugModel namespace? Noticing that the SmugMugModel tests are much more extensive.

@AlexGhiondea
Copy link
Owner

SmugMugModel is the v1 version -- that one is definitively more feature rich!

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