-
Notifications
You must be signed in to change notification settings - Fork 7
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
root v2 at this point #38
Open
ivinjabraham
wants to merge
37
commits into
amfoss:master
Choose a base branch
from
ivinjabraham:decouple-shuttle
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+2,098
−3,464
Open
Changes from 1 commit
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
e6224dd
remove soon-to-be outdated information in readme
ivinjabraham 4f4ee27
remove shuttle dependency
ivinjabraham 3c01c3a
reduce codebase to basic working web server template
ivinjabraham c3c58d1
remove migrations
ivinjabraham 4f12588
add database backup to gitignore
ivinjabraham cec9d58
remove unused state from router
ivinjabraham d09e6a5
ignore .env files
ivinjabraham ec44d94
add migration to create initial tables for member, streaks and attend…
ivinjabraham bd5cd88
remove shuttle workflows
ivinjabraham 522ded8
rename db module to models in order to better reflect its contents
ivinjabraham a16c83b
add graphql interface back to the server
ivinjabraham e8d341a
minor refinements to loading env variables, db pool configuration and…
ivinjabraham e7f43c7
uncomment scheduled task
ivinjabraham 7ca67ea
rename scheduled_task to daily_task
ivinjabraham b3f1842
rename reference to db module to models, was left out earlier
ivinjabraham 6201d06
use kolkata timezone instead of local time
ivinjabraham 0ba8c00
rewrite midnight attendance records insertion and required models
ivinjabraham 86ec4e0
rewrite attendance record insertion and updation logic
ivinjabraham 7a9a6ee
remove leaderboard and active projects feature
ivinjabraham 867ac2d
documentation changes and additions
ivinjabraham 7762c85
ignore .log files
ivinjabraham 3518e18
rewrite graphql interface
ivinjabraham ba21373
remove tests
ivinjabraham e6657da
add re-exports for less verbose imports
ivinjabraham 76ac325
add BIND_ADDRESS env variable to avoid hardcoding address
ivinjabraham 3ba685d
improve documentation in main.rs
ivinjabraham e2207f7
refactor main.rs to be more high-level by abstracing away its processes
ivinjabraham 2d3c497
rename endpoints for more clarity
ivinjabraham 3be56a5
complete todo to handle error when failing to fetch members
ivinjabraham 3512e6c
add documentation to models
ivinjabraham 127cb5e
fix graphiql missing graphql schema
ivinjabraham ffd6879
update documentation for root
ivinjabraham 173b39c
add reset streak mutation
ivinjabraham 7a804dc
add hmac verification to mark attendance
ivinjabraham 8690495
remove shuttle workaround in main.rs
ivinjabraham 7655247
fix incorrect query in mark attendance and remove time fields from it…
ivinjabraham 356e508
run cargo fmt
ivinjabraham File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
same thing here, imo it should be rewritten before a possible merge. @swayam-agrahari whats your thoughts?
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 we can merge the pr first then later we can work on the leaderboard, I haven't had time to complete the frontend so it will be done in the mean time and others are busy with gsoc so don't thing its a urgency right now!