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.

Sign up for GitHub

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

Jump to bottom

Borrow Count homepage graph should redirect to /stats #3229

Closed
mekarpeles opened this issue Mar 20, 2020 · 2 comments
Closed

Borrow Count homepage graph should redirect to /stats #3229

mekarpeles opened this issue Mar 20, 2020 · 2 comments
Assignees
@ScarlettRed
Labels
Affects: UI Issues with the web site's user interface. [managed] Good First Issue Easy issue. Good for newcomers. [managed] Lead: @mekarpeles Issues overseen by Mek (Staff: Program Lead) [managed] Priority: 3 Issues that we can consider at our leisure. [managed] Type: Bug Something isn't working. [managed]

Comments

@mekarpeles
Copy link
Member

@ScarlettRed would you be willing to open a PR to fix this?

I also checked only today. The e-books burrowed graph is working correctly now, but there is one more issue related to this. The link given with the burrowed graph is redirecting to some unexpected page redirecting page's URL is: https://openlibrary.org/subjects/in_library#ebooks=true This page doesn't show any information regarding e-books burrowed.

Originally posted by @ScarlettRed in #3217 (comment)
@mekarpeles mekarpeles added Affects: UI Issues with the web site's user interface. [managed] Good First Issue Easy issue. Good for newcomers. [managed] Priority: 3 Issues that we can consider at our leisure. [managed] Home Page labels Mar 20, 2020
@ScarlettRed
Copy link
Contributor

@mekarpeles sure thing! I will open a PR to fix this issue.

@mekarpeles mekarpeles mentioned this issue Mar 23, 2020
Closed
@xayhewalo xayhewalo added the Type: Bug Something isn't working. [managed] label Mar 25, 2020
@BrittanyBunk BrittanyBunk mentioned this issue Mar 29, 2020
Closed
@xayhewalo xayhewalo added the Lead: @mekarpeles Issues overseen by Mek (Staff: Program Lead) [managed] label Apr 7, 2020
@mekarpeles
Copy link
Member Author

I looks like this has been implemented #3307

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ScarlettRed ScarlettRed

Labels
Affects: UI Issues with the web site's user interface. [managed] Good First Issue Easy issue. Good for newcomers. [managed] Lead: @mekarpeles Issues overseen by Mek (Staff: Program Lead) [managed] Priority: 3 Issues that we can consider at our leisure. [managed] Type: Bug Something isn't working. [managed]
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mekarpeles @xayhewalo @ScarlettRed