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

💄 Add global information #35

Merged
merged 3 commits into from
Apr 20, 2019
Merged

💄 Add global information #35

merged 3 commits into from
Apr 20, 2019

Conversation

christopherkade
Copy link
Collaborator

@christopherkade christopherkade commented Apr 18, 2019

  • Added a landing view with a catchphrase
  • The number of members in the organisation are now displayed
  • The number of repositories are also displayed
  • Updated styles to follow the BEM convention

(Styles are a work in progress, the goal being to display the data)

Github Insights must be updated to retrieve the number of PRs made by members of the organisation in order to move forward. (See Zenika/github-insights#8)

Screenshot 2019-04-20 at 00 27 59

The number of members and repositories are now displayed alongside a landing page #4
@tbetous
Copy link
Contributor

tbetous commented Apr 18, 2019

Nice design ! 👍

@christopherkade
Copy link
Collaborator Author

Nice design ! 👍

Thank you ! It's far from what the final product will be, I just threw it together to avoid having something empty for now. We'll need to agree on a final design eventually but until then the MVP will probably stay simple.

Made the display nicer, added countup.js to animate our stats #4
@frinyvonnick
Copy link
Contributor

Is this still work in progress @christopherkade 😄 ?

@christopherkade
Copy link
Collaborator Author

Is this still work in progress @christopherkade 😄 ?

It is since we are missing the number of PRs done by our members (#4). It would involve updating the Github Insights project first and then updating the current view.

@frinyvonnick
Copy link
Contributor

Maybe we could postpone this part ? If so could I review this PR ?

@christopherkade
Copy link
Collaborator Author

Sounds good to me, of course, go ahead ! 😄

Copy link
Contributor

@frinyvonnick frinyvonnick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work @christopherkade 👏 I left a few comments 😉

Switched to react-countup, updated html tags for accessibility
@christopherkade christopherkade merged commit 77a711e into zenika-open-source:master Apr 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants