Added cp leaderboard queries and mutations #97
Conversation
|
Your commit messages could use some work; refer https://www.conventionalcommits.org/en/v1.0.0/ and https://www.simplethread.com/what-makes-a-good-git-commit/. |
|
Rewrite the commits and fix the failing workflow and you're good to go. |
|
@ivinjabraham ok thanks👍 |
AadarshM07
force-pushed
the
master
branch
2 times, most recently
from
02b2cd0 to
9ca8e7e
Compare
|
@ivinjabraham I have updated the PR with the requested changes |
| @@ -5,3 +5,4 @@ Secrets*.toml | |||
| backups/ | |||
| .env | |||
| *.log | |||
|
|
|||
|
|
||
| CREATE TABLE IF NOT EXISTS leetcode_stats ( | ||
| id SERIAL PRIMARY KEY, | ||
| member_id INT NOT NULL, |
|
|
||
| CREATE TABLE IF NOT EXISTS codeforces_stats ( | ||
| id SERIAL PRIMARY KEY, | ||
| member_id INT NOT NULL, |
| ALTER TABLE leetcode_stats ADD CONSTRAINT leetcode_stats_member_id_key UNIQUE (member_id); | ||
| ALTER TABLE codeforces_stats ADD CONSTRAINT codeforces_stats_member_id_key UNIQUE (member_id); |
There was a problem hiding this comment.
Why are we altering it here? We can add the constraint directly during creation
| Ok(members) => { | ||
| for member in members { | ||
| // Fetch LeetCode username | ||
| let leetcode_username = sqlx::query_as::<_, LeetCodeStats>( |
There was a problem hiding this comment.
Change the variable name, the SQL will return an entire row from the leetcode table
| Ok(_) => { | ||
| println!("LeetCode stats updated for member ID: {}", member.member_id) | ||
| } | ||
| Err(e) => eprintln!( | ||
| "Failed to update LeetCode stats for member ID {}: {:?}", |
There was a problem hiding this comment.
Migrate from print statements to the tracing/logging library
| Ok(_) => println!( | ||
| "Codeforces stats updated for member ID: {}", | ||
| member.member_id | ||
| ), | ||
| Err(e) => eprintln!( | ||
| "Failed to update Codeforces stats for member ID {}: {:?}", | ||
| member.member_id, e |
There was a problem hiding this comment.
Migrate from print statements to the tracing/logging library
| } | ||
|
|
||
| // Fetch Codeforces username | ||
| let codeforces_username = sqlx::query_as::<_, CodeforcesStats>( |
There was a problem hiding this comment.
Change the variable name, the SQL will return an entire row from the codeforces table
This adds the necessary queries and mutations required for the cp leaderboard . It includes:
All functionalities have been tested using the GraphiQL playground