-
-
Notifications
You must be signed in to change notification settings - Fork 836
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 links to troubleshooting documentation in exception messages #1248
Comments
Instead of using bit.ly links, could be not just create pages on autofac.org that redirect to places in our docs? Pretty trivial to do I believe with a custom Jekyll layout file, and a |
Yeah, I don't have a strong preference of how it gets done, I was just interested in the idea of being able to provide direct doc links to common issues. Though, admittedly, I feel like an actual 302 is somewhat better than an HTML-based meta redirect, simply because non-browsers can deal with it a little easier (if that's a use case). |
I've taken a look at options for the shortener. bit.ly will provide the functionality we need 'for free', including creating links with custom path content, e.g. http://bit.ly/autofac-scope-docs (link works). The only really obvious downside is that you can't add other users to manage those links, so we would have to share creds (not super ideal, but manageable). One alternative that shows promise is setting up 'Exact Redirects' on readthedocs itself. We should be able to set up https://autofac.rtfd.io/help/di-exception (or similar) to point where we want, and that would end up on the right page. Benefit of this approach is that we don't need to create any new accounts or anything, and our links stay under the same domain as our docs. Plus, with their Only issue with the ReadTheDocs approach is that...it doesn't work on our project, but does appear to work on my personal fork. I've raised an issue with them to ask why that might be. If the answer is straightforward, this approach is probably my preference. |
Oh, I didn't know about the native RTD version. Keep me posted on what you find, I like that one, too. |
According to the ReadTheDocs people, our docs are set up as a translation of a different set of docs, |
👀 UHHHHHH noooooo that doesn't seem familiar at all. However, I do see on the dashboard that's how it's set. Maybe something in the config file? I'll look around a bit. |
I'm not seeing anything like that anywhere in the config but... @alexmg @nblumhardt could I do recall there was a lot of shuffle when they moved from |
To quote from the RTD issue:
|
Yes, let's decouple that. |
My only concern might be that any existing references in other people's blog posts or similar generated content will break in that event? |
If possible we sort of want to invert the relationship. |
We might have some link breakages, yeah, but we may be able to set up some sort of other project and redirect or something. Given we're somewhat stuck on other RTD features because of this, I'd somewhat rather rip the bandage off and figure out how to fix it than try to leave it alone and try to add more workarounds to the mix. |
The DNS has two A records pointing to Cloudflare IP addresses. I think only @nblumhardt has access to the configuration in Cloudflare. |
Odd! I've never encountered |
Ah, and 👋 hey all, hope you're all doing well! 😎 |
Right, turns out someone once indicated that our site was a translation of there's (somehow?); don't entirely understand, but you can see the details here: readthedocs/readthedocs.org#8134 (comment) I'm going to suggest we give the original forker(?) the chance to fix it so redirects don't break, but if that doesn't pan out we'll just pull the plug on the connection between the two. |
I think that's a reasonable plan. Let me know if there's anything I can help with. What a cluster. |
Right, so this has been an entertaining morning, during which time our docs were entirely broken for an hour or two. Basically, that original owner of the I then claimed the project name I've put in place a redirect that will take an existing This isn't ideal, but at least existing links from StackOverflow answers won't break. Feel free to read the blow-by-blow: readthedocs/readthedocs.org#8134 (comment). As @tillig said, a bit of a cluster. |
😩 It's always something. Good work tracking this down! I think we can live with the simple main page you have there. Maybe at some point we can figure out how to add something like this redirect plugin to the site to handle the main page. In any case, is call this a win. |
@tillig, do you know who controls the DNS for Otherwise |
It's @nblumhardt - he mentioned a while back if we ping him for something (ping!) he might be able to help out. |
Done! Might take a short while to propagate - let me know if it needs any tweaking 👍 |
Thanks @nblumhardt, looks good. 👍 |
I think everything here is wrapped up and merged. This should show up in exception messages in the next release. |
Problem Statement
Despite having reasonably clear error messages for our exceptions, there continues to be an... inordinate... number of StackOverflow questions that are all effectively titled "Autofac DependencyResolutionException" (which isn't a question and largely indicates folks haven't actually tried fixing it, but I digress). It's unclear whether the message is still confusing to people or whether it's a case of "TL;DR."
Desired Solution
It may be helpful to:
bit.ly
links to these pages.Alternatives You've Considered
Additional Context
It doesn't have to be
bit.ly
, but:The text was updated successfully, but these errors were encountered: