-
Notifications
You must be signed in to change notification settings - Fork 4
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 Citizen profile page #11
Conversation
pages/profile/[ethAddress].tsx
Outdated
colorGradient.addColorStop(1, 'rgba(213, 163, 152, 0.8)') | ||
|
||
// Fetch data from datasets repo | ||
const sourceCredFileUrl: string = `https://raw.githubusercontent.com/nation3/nationcred-datasets/f9e5b8d30b969815a918de619fa4e89a1c800ce3/data-sources/sourcecred/output/sourcecred-${profile.ethAddress}.csv` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@johnmark13 Using the data from your PR here. Once merged, we can update this URL.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is super cool. The chart colors at the moment slightly limit legibility/determining different streams, but that seems a small thing for now.
@johnmark13 Yes, we can easily adjust the colors. Do you have suggestions for which colors to use? 😄 Also note that you can hover points in the chart to see details: |
@johnmark13 Added Dework chart: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
Maybe we can also add pagination or a custom search bar on the directory / default page for better UX in the future.
@okhaimie-dev Great ideas! Added those here: |
Why the "Loading profile..." placeholder? |
@xPi2 Yes, a placeholder. Work in progress 😄 |
Dework Task
https://app.dework.xyz/nation3/nationcred-p?taskId=246b633c-5075-4ba4-a6d1-e8e62bd96ffc
Screenshots