Skip to content
This repository has been archived by the owner on Jan 31, 2024. It is now read-only.

Jwen/scoreboard basic #44

Merged
merged 13 commits into from
Mar 9, 2022
Merged

Jwen/scoreboard basic #44

merged 13 commits into from
Mar 9, 2022

Conversation

jiabaow
Copy link
Collaborator

@jiabaow jiabaow commented Mar 8, 2022

Screen Shot 2022-03-08 at 21 50 08

Normally we should be able to see the profile photos between the ranking and the names. But I don't know why it doesn't work with my hard-coded data. I will leave them like this until we start loading data from firebase.

@jiabaow jiabaow self-assigned this Mar 8, 2022
@jiabaow jiabaow marked this pull request as draft March 8, 2022 20:52
@codeclimate
Copy link

codeclimate bot commented Mar 9, 2022

Code Climate has analyzed commit 2049cd0 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 100.0% (0.0% change).

View more on Code Climate.

@jiabaow jiabaow mentioned this pull request Mar 9, 2022
2 tasks
@jiabaow jiabaow marked this pull request as ready for review March 9, 2022 14:56
@jiabaow jiabaow requested a review from Tsathogguaa March 9, 2022 14:56
@laurislopata laurislopata self-requested a review March 9, 2022 15:07
@Tsathogguaa Tsathogguaa merged commit bcc9d6a into main Mar 9, 2022
@Tsathogguaa Tsathogguaa deleted the jwen/scoreboard-basic branch March 9, 2022 15:21
@jiabaow jiabaow linked an issue Mar 11, 2022 that may be closed by this pull request
2 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement the scoreboard (#15)
3 participants