-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
fixed wrong doc link #1449
fixed wrong doc link #1449
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1449 +/- ##
=======================================
Coverage 33.19% 33.19%
=======================================
Files 41 41
Lines 4950 4950
Branches 1135 1135
=======================================
Hits 1643 1643
Misses 3177 3177
Partials 130 130
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
I just looked on the deployed version of the website and it seems like the link is working (https://microsoft.github.io/autogen/docs/Use-Cases/agent_chat). In what environment did you notice the broken link? |
Hey @jackgerrits
Well I think I found the issue. The link on the website redirects me to https://microsoft.github.io/autogen/docs/Use-Cases/agent_chat Which is greate. But directly requesting the page https://microsoft.github.io/autogen/docs/Use-Cases/agent_chat Does this also happen to you? |
Ah thanks for explaining the issue! If you reference the |
Co-authored-by: Jack Gerrits <[email protected]>
Thanks for finding this, and pushing through this fix! |
* fixed wrong doc link * directly referenced file Co-authored-by: Jack Gerrits <[email protected]> * Update website/docs/Use-Cases/agent_chat.md --------- Co-authored-by: Jack Gerrits <[email protected]> Co-authored-by: Chi Wang <[email protected]>
This link points to nothing. I think the right link should point to this