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

Add use case: Migrate my blog comments from Disqus to Github Issues #102

Closed
lazywinadmin opened this issue Apr 8, 2019 · 3 comments · Fixed by #116
Closed

Add use case: Migrate my blog comments from Disqus to Github Issues #102

lazywinadmin opened this issue Apr 8, 2019 · 3 comments · Fixed by #116
Labels
good first issue If you're new to the project (or to OSS in general) and you'd like to contribute, try this one. help wanted Anyone in the community is welcome to do this work up for grabs Anyone in the community is welcome to do this work

Comments

@lazywinadmin
Copy link
Contributor

Not sure if you would be interested to add this into your Readme examples/use cases.
I used your module to migrate all my disqus blog comments to Github issues. It is using Utterances-Bot to create new issues and display existing comments.

This is how i found the issue fixed by my PR #76
Thanks for this tool, I'll continue to contribute if i see other bugs.

Here is my blog post where i talked about it: https://lazywinadmin.com/2019/04/moving_blog_comments.html

@HowardWolosky
Copy link
Member

Thanks @lazywinadmin -- It's great to see how helpful the module was for you. That's a very detailed/well-written article as well. If you'd like to create a PR to add an update to USAGE.md that references your blog post as a more advanced example, I'd approve it.

One thing I did notice in there is that you disabled telemetry as part of your script execution. That's totally fine -- that configuration option is there to provide each user that choice. I'm simply curious why you made that choice -- do you generally disable telemetry for all the products that you use, or did you do it for this tool specifically for some other reason? I ask, because if there's a way to improve the telemetry in this project so that users don't feel a need to disable it, that will result in an overall net postive.

The telemetry is helpful for knowing that there are people out there, using it. There's certainly more motiviation for continued maintenance and improvement of the project when there's a way to measure its audience.

@lazywinadmin
Copy link
Contributor Author

Thank you

I agree with the telemetry approach and I'm not against it.
I think i turned it off because the first time i ran New-GithubIssue it took more than 20 seconds to add the issue. So i just disabled it since I had to create many issues and comments for my blog comment migration.

I just retested.. I did not realize that the 20 seconds seem to be only for the first call.
Other issues are created under 1/2 seconds.

Here is a screenshot of that first issue creation (it took +20sec)
image

@HowardWolosky
Copy link
Member

Thanks @lazywinadmin. That first-run experience is currently expected, as the first-run experience downloads 3 NuGet packages to get the necessary binaries to submit the telemetry through Application Insights. It then caches them for that PowerShell session, and tells you how to permanently cache them (by storing them in your module directory OR a directory of your choice and then setting the AssemblyPath configuration value).

I've created #104 to track a feature to remove that requirement entirely by reverse-engineering the web request that the dll's use to communicate with Application Insights.

But yes, that is an unfortunate first-run experience in order for telemetry to work.

@HowardWolosky HowardWolosky added good first issue If you're new to the project (or to OSS in general) and you'd like to contribute, try this one. help wanted Anyone in the community is welcome to do this work up for grabs Anyone in the community is welcome to do this work labels Apr 15, 2019
lazywinadmin added a commit to lazywinadmin/PowerShellForGitHub that referenced this issue Jun 26, 2019
See microsoft#102 
I used this module to migrate my blog comments to Github Issues
HowardWolosky pushed a commit that referenced this issue Jul 29, 2019
@lazywinadmin used this module to migrate his blog comments to Github Issues.
That advanced example is now referenced in the USAGE documentation.

Resolves #102
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue If you're new to the project (or to OSS in general) and you'd like to contribute, try this one. help wanted Anyone in the community is welcome to do this work up for grabs Anyone in the community is welcome to do this work
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants