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

Bar Chart Update (Billing/Analytics) #1145

Closed
HollyJoyPhillips opened this issue May 4, 2020 · 2 comments · Fixed by #1168 or #1216
Closed

Bar Chart Update (Billing/Analytics) #1145

HollyJoyPhillips opened this issue May 4, 2020 · 2 comments · Fixed by #1168 or #1216
Assignees
@duong-se
Labels
enhancements Relates to a feature enhancement front-end Relates to front end issues high-priority Issues scheduled for upcoming major release marketplace Relates to the Marketplace
Milestone
Developer Analyti...

Comments

@HollyJoyPhillips
Copy link
Contributor

Summary: We now have the billing endpoint to populate the ‘Services’ bar chart on the Billing
tab in the Developers Portal Analytics

Endpoint: https://dev.platformmarketplace.reapit.net/trafficevents/swagger/index.html
/billing

Location: https://dev.marketplace.reapit.cloud/developer/analytics/billing

The chart is based on the ‘grossAmount’ for each month.

  • The first month should be the month the developer was created (/developers/{id} ‘created’)
  • Query the /billing endpoint, supply the ApplicationIds (will need to supply all apps for that developer - to get the ID's you will need to query 'Apps by DeveloperId')
  • 'DateFrom' should be the month the developer was created
  • 'DateTo' should be the current month
  • The endpoint will return a ‘requestsByPeriod’ which will include those months.
  • Use the ‘periodName’ to label each bar (should include the year)
  • Display the ‘grossAmount’ for each bar
@HollyJoyPhillips HollyJoyPhillips added marketplace Relates to the Marketplace enhancements Relates to a feature enhancement front-end Relates to front end issues labels May 4, 2020
@willmcvay willmcvay added the high-priority Issues scheduled for upcoming major release label May 5, 2020
@duong-se duong-se self-assigned this May 5, 2020
duong-se added a commit that referenced this issue May 6, 2020
@duong-se
refactor: #1145 refactor billing bar chart
7555efc 
Changes
- Rename to developer-analytics
- Move to specific folder to manage easily
@duong-se duong-se mentioned this issue May 6, 2020
Merged
duong-se pushed a commit that referenced this issue May 6, 2020
refactor: #1145 refactor billing bar chart (#1154)
c9a2dac 
refactor: #1145 refactor billing bar chart (#1154)
@duong-se duong-se mentioned this issue May 7, 2020
Merged
@duong-se duong-se linked a pull request May 7, 2020 that will close this issue
feat: #1145 integrate api for billing bar chart #1168
Merged
@HollyJoyPhillips
Copy link
Contributor Author

Hi @tanphamhaiduong Just a small tweak please, can you add a pound sign (£) for the hover over, so it will be £.6.88

image

@duong-se
Copy link
Contributor

As discussion with @HollyJoyPhillips we'll add like the image below.
image

duong-se added a commit that referenced this issue May 12, 2020
@duong-se
fix: #1145 minor fix for lY legend of bar chart
f4f1310 
resolve #1145
Changes
- Minor fix for bar chart Y legend
@duong-se duong-se mentioned this issue May 12, 2020
Merged
duong-se added a commit that referenced this issue May 12, 2020
@duong-se
fix: #1145 minor fix for Y axis's legend of bar chart
80a7512 
resolve #1145
Changes
- Minor fix for bar chart Y legend
duong-se added a commit that referenced this issue May 12, 2020
@duong-se
fix: #1145 minor fix for Y axis's legend of bar chart
23f6f3d 
resolve #1145
Changes
- Minor fix for bar chart Y legend
duong-se added a commit that referenced this issue May 12, 2020
@duong-se
feat: #1145 change grossAmount to netAmount to show on bar chart
a1df9d4 
Changes
- Change grossAmount to netAmount
- Fix test
duong-se pushed a commit that referenced this issue May 13, 2020
fix: #1145 minor fix for Y axis's legend of bar chart and change to n…
63ae65d 
…etAmount instead of grossAmount (#1216)

Resolve #1145
Changes
- Minor fix for bar chart Y legend
- Change grossAmount to netAmount
- Fix test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@duong-se duong-se

Labels
enhancements Relates to a feature enhancement front-end Relates to front end issues high-priority Issues scheduled for upcoming major release marketplace Relates to the Marketplace
Projects
None yet
Milestone
Developer Analytics v2.0
3 participants
@willmcvay @duong-se @HollyJoyPhillips