-
Notifications
You must be signed in to change notification settings - Fork 848
feat(ui): show <owner>/<repo>
in page title
#303
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried it and seems to be working correctly, thanks a lot for your contribution @amaank404
Before merging I would like to clear out those two comments and we should be fine!
<owner>/<repo>
in page title
alright, I've tested the final changes, all should be in order now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and it works very nice!
Thank you very much, merging this
It was addressed with @cyclotruc review
Modified the template files to include the repo in the title if there is.
Modified the utils to read the title and automatically set it in the document
Modified the query processor to include a 'short_repo_url' field in the context.
Closes: #271