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

Sharing link for government revenue (including grants), % of GDP broken #453

Open
jamesharle opened this issue Oct 30, 2018 · 1 comment
Assignees
Labels
Milestone

Comments

@jamesharle
Copy link
Contributor

I'm submitting a ...

Check one of the following options with "x" and add the appropriate label to the issue as well


[X] Bug report 
[ ] Regression (behaviour that used to work and stopped working in a new release)

Describe the Issue
When user configures global picture/government finance graph to government revenue (including grants), % of GDP, the share this chart button (as configured) produces a shortened link that directs user to a different /government finance indicator (government revenue per person (including grants), 2011 PPP$)

To Reproduce
Replace the content below with the steps to reproduce the behaviour.

  1. Go to 'http://data.devinit.org/global-picture/government-finance'
  2. Click on 'government revenue (including grants), % of GDP' on dropdown menu
  3. Click on 'share this chart'
  4. Copy link
  5. Follow link
  6. See error

Expected Behaviour
The correct behaviour would see the button producing a link that redirects to the correct indicator (government revenue (including grants), % of GDP)

Screenshots/GIF
If applicable, add screenshots or a GIF to help explain your problem.

Desktop (please complete the following information):

  • OS: [Windows 7]
  • Browser [Chrome]
  • Version [69.0.3497.100]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Environment/Server:


- [ ] Production
- [ ] Staging
- [ ] Test
- [ ] Development

Additional Context
Add any other context about the problem here.

Requested Delivery Date: dd/MMM/YYYY

@edwinmp edwinmp self-assigned this Nov 19, 2018
@edwinmp edwinmp added this to the January 2019 milestone Jan 7, 2019
@edwinmp
Copy link
Contributor

edwinmp commented Jan 28, 2019

Hi @jamesharle ...this should be fixed on staging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants