Skip to content

Update migrations #23

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 26 commits into
base: main
Choose a base branch
from
Open

Update migrations #23

wants to merge 26 commits into from

Conversation

VishakhaSainani-Josh
Copy link
Collaborator

@VishakhaSainani-Josh VishakhaSainani-Josh commented Jun 30, 2025 •
edited
Loading

Update migrations-

  1. Add Column contributors url in repositories table
  2. . Allow null values for contribution id in transaction table. As transaction contains both redeemed and gained transactions so contribution id will be null for redeemed one.
  3. Need to add column github_event_id in contributions table to use the gh archive contribution id for querying instead of usernames.
  4. Set Not null constraint for contributors_url in repositories table
  5. Set Not null constraint for github_event_id in contributions table.

Please review all migration related changes in this PR

@VishakhaSainani
create contribution fetching for all users in the system from bigquer…
3834565 
…y service (in future this will be a cron job that will be scheduled to run daily)
@VishakhaSainani
Merge branch 'feat/db_migrations' into feat/contribution-service
1c7b715
@VishakhaSainani
Merge branch 'fix/login-timestamp-type' into feat/contribution-service
284fdaf
@VishakhaSainani
use userid while adding contribution
d3d91a4
@VishakhaSainani
get id and score from contribution_score
1c1135f
@VishakhaSainani
fix date format
f94000e
@VishakhaSainani
refactor processfetchedcontribution function and update http client h…
a17215c 
…andling while fetching repositoryDetails
@VishakhaSainani
implement users five recent contributions
de13e7a
@VishakhaSainani
fetch all contributions for the user
f7e8fa9
@VishakhaSainani
remove unnecessary lines
887faa4
@VishakhaSainani
pass http client in service struct
bb68dfa
@VishakhaSainani
add contributors_url in repository table
e5a4d0b
@VishakhaSainani
Merge branch 'db/migration/add-contributors-url' into feat/contributi…
a8e5768 
…on-service
@VishakhaSainani
fetch and contributors url for repository
5773245
@VishakhaSainani
remove fetch five recent contributions for the user- this could be ha…
f365171 
…ndled in frontend
@VishakhaSainani
allow null contribution id for redeemed transactions
6c3a6b6
@VishakhaSainani
Merge branch 'db/migration/allow-null-contributionid' into db/migrati…
bdc0c0a 
…on/add-gh-event-id
@VishakhaSainani
add column gh-event-id to use gh-event-id for contributions instead o…
71d8a4a 
…f usernames
@VishakhaSainani
seperate and use one alter table per migrations
165fec8
@VishakhaSainani
set column contributors as not null in seperate migration file
82fb23a
@VishakhaSainani
remove unnecessary lines of code
7980470
@VishakhaSainani
Merge branch 'db/migration/add-contributors-url' into db/migration/ad…
80bab9c 
…d-gh-event-id
@VishakhaSainani
Merge branch 'db/migration/allow-null-contributionid' into db/migrati…
6194eb4 
…on/add-gh-event-id
@VishakhaSainani
rename gh_event_id to github_event_id to ensure consistency across names
5d8cb59
@VishakhaSainani
set not null github-event-id
230cd4e
@VishakhaSainani
remove default value 0 for column contribution id in table transaction
9e7f3a8
@VishakhaSainani-Josh VishakhaSainani-Josh changed the title Update migrations- add gh event id Update migrations Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Suhaan-Bhandary-Josh Suhaan-Bhandary-Josh Awaiting requested review from Suhaan-Bhandary-Josh

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@VishakhaSainani-Josh @VishakhaSainani