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

Fix developers section info #132

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Fix developers section info #132

wants to merge 2 commits into from

Conversation

mecm1993
Copy link

Currently, the dev information regarding the repositories and forks is not showing properly,
Screen Shot 2019-06-10 at 1 08 58 AM

I made a minor modification in the user query so now it takes in consideration the information of the repositories and forks,
Screen Shot 2019-06-10 at 1 08 50 AM

Also, I added the column/property description for Users due to it was missing and the following information as well in the DeveloperCard component.

 - Minor modification in user query
 - New description property in User
 - Updated user data
 - Modifications in services to take in consideration new property
 - Description text available when hovering the dev card
@mecm1993 mecm1993 changed the title Fix developers info Fix developers section info Jun 10, 2019
@DannyFeliz
Copy link
Member

DannyFeliz commented Jun 10, 2019

Nice 🎉
@rmariuzzo could you take a look?

@hectoraristy
Copy link
Contributor

LGTM, Excellent job @mecm1993

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