Closed kamaladenalhomsi closed 4 years ago
Nice work, but let me give some suggestions:
project-name/pull/number
) for each PR, the username should be used to group them togetherFYI, you must run npm run fetch-prs
several times to fill the database enough to get enough users/prs to see in the front-end.
seems that you have another version of the website, that's what website looks like in my device
ops, I forgot to disable the darkreader (dark mode for all websites extension) :man_facepalming:
Please, check the other suggestions.
@paulodiovani ok, I am going to work on it
@paulodiovani can you merge this for now (I am seeking for 2 additional pull requests 😅)
@kamaladenalhomsi sorry man, But we need at least to list the PRs links like it was before. It is just showing the username many times.
I am seeking for 2 additional pull requests
same :boat: here
@paulodiovani I replaced the "Go to PR" with the "project-name/pull/number", other suggestions will be done in a separate PR.
I noticed the website design is poor, so in this PR, I tried to improve the design as much as I can, this design is heavily inspired by the hacktoberfest website,
@paulodiovani if the design is ok, I hope you merge ASAP, so I can complete working in on what you mentioned on #8 (I will complete working on separate PRs, still need two PRs to for the hacktoberfest 😅 )