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

As a ‘Developer’, I want to see a table showing installations for each of my apps #172

Closed
HollyJoyPhillips opened this issue Feb 3, 2020 · 4 comments · Fixed by #186
Closed

As a ‘Developer’, I want to see a table showing installations for each of my apps #172

HollyJoyPhillips opened this issue Feb 3, 2020 · 4 comments · Fixed by #186
Assignees
@vuhuucuong
Labels
enhancements Relates to a feature enhancement feature New feature or request front-end Relates to front end issues marketplace Relates to the Marketplace
Milestone
Foundations Alpha...

Comments

@HollyJoyPhillips
Copy link
Contributor

HollyJoyPhillips commented Feb 3, 2020 •
edited
Loading

Given that I am a ‘Developer’, when I access the ‘Analytics’ tab from the Developer portal, I want to then see a paginated list of installations for each of my apps.

Location: https://dev.marketplace.reapit.com/developer/analytics

Summary: We already have this functionality on the https://dev.marketplace.reapit.com/developer/analytics page but instead of selecting which app you want to see the results of, it should just automatically show all apps in a table.

Task: To update the analytics page and add a column ‘App Name’ to the existing table and include the associated ‘App Name’:

Page Layout

  • Heading: Change from 'Analytics' to Dashboard' (only the page heading not the nav bar)
  • Background colour: White

Table

  • Heading: Installations
  • Sub text: The installations table below shows the individual installations per client with a total number of installations per app
  • Show total count of ‘CURRENT’ installations (installations that do not have a ‘terminatedOn’ date). Example: Total number of current installations for App Name 1: 8
  • Order results by ‘Date Installed’ newest first
  • Show max 5 results
  • Table should be paginated

See example:
image

Mock up:
image
@HollyJoyPhillips HollyJoyPhillips added feature New feature or request marketplace Relates to the Marketplace enhancements Relates to a feature enhancement labels Feb 3, 2020
@HollyJoyPhillips HollyJoyPhillips mentioned this issue Feb 3, 2020
Closed
@HollyJoyPhillips HollyJoyPhillips added front-end Relates to front end issues back-end Relates to back end issues and removed front-end Relates to front end issues labels Feb 4, 2020
@vuhuucuong vuhuucuong self-assigned this Feb 5, 2020
vuhuucuong added a commit that referenced this issue Feb 5, 2020
@vuhuucuong
feat: #172 Add table showing installation of app for developer page
2e8682b
vuhuucuong added a commit that referenced this issue Feb 5, 2020
@vuhuucuong
feat: #172 Remove import css
53caea5
@vuhuucuong vuhuucuong mentioned this issue Feb 5, 2020
Merged
vuhuucuong added a commit that referenced this issue Feb 6, 2020
@vuhuucuong
feat: #172 add some comment and type hints
33eba01
vuhuucuong added a commit that referenced this issue Feb 6, 2020
@vuhuucuong
feat: #172 remove unnecessary deps
239de1d
vuhuucuong added a commit that referenced this issue Feb 6, 2020
@vuhuucuong
feat: #172 separate functions for readable
b227cfa
vuhuucuong added a commit that referenced this issue Feb 6, 2020
@vuhuucuong
feat: #172 add table showing installation of app for developer (#186)
119b2f9 
* feat: #172 Add table showing installation of app for developer page

* feat: #172 Remove import css

* feat: #172 add some comment and type hints

* feat: #172 remove unnecessary deps

* feat: #172 separate functions for readable
@vuhuucuong
Copy link
Contributor

vuhuucuong commented Feb 6, 2020 •
edited
Loading

Hi @HollyJoyPhillips , quick note when QA this and #171 #173 #174
The styles may not look the same with the mockup. But since other guys are coding on #171 #173 #174 in the same file, we can fix that later to avoid conflicting. Just raise an issue to fix styling after those above issues are done if you feel necessary. Thanks :)

@HollyJoyPhillips
Copy link
Contributor Author

@vuhuucuong Thanks for the heads up regarding the UI. The installations table is great. Just a minor change to the wording but might be worth including in another PR instead.
It should say: The installations table below shows the individual installations per client with a total number of installations per app

@vuhuucuong
Copy link
Contributor

Ah, thanks. I included a fix in this PR #190
This should be fixed after that PR's merged

@HollyJoyPhillips
Copy link
Contributor Author

Great stuff, thank you

@HollyJoyPhillips HollyJoyPhillips added front-end Relates to front end issues and removed back-end Relates to back end issues labels Feb 10, 2020
nphivu414 pushed a commit that referenced this issue Apr 29, 2020
@zlatanpham
[CLD-471] fix file upload size (#172)
960ad41
nphivu414 pushed a commit that referenced this issue Apr 29, 2020
@vuhuucuong
feat: #172 add table showing installation of app for developer (#186)
f7a1539 
* feat: #172 Add table showing installation of app for developer page

* feat: #172 Remove import css

* feat: #172 add some comment and type hints

* feat: #172 remove unnecessary deps

* feat: #172 separate functions for readable
@snyk-bot snyk-bot mentioned this issue Nov 25, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vuhuucuong vuhuucuong

Labels
enhancements Relates to a feature enhancement feature New feature or request front-end Relates to front end issues marketplace Relates to the Marketplace
Projects
None yet
Milestone
Foundations Alpha Developer Release
Development

Successfully merging a pull request may close this issue.

feat: #172 add table showing installation of app for developer reapit/foundations
2 participants
@vuhuucuong @HollyJoyPhillips