Skip to content

fix(server): sync GitHub info upon login - #59

Merged
MingcongBai merged 1 commit into
masterfrom
login-fix
Apr 6, 2026
Merged

fix(server): sync GitHub info upon login#59
MingcongBai merged 1 commit into
masterfrom
login-fix

Conversation

@xtexx

@xtexx xtexx commented Apr 4, 2026

Copy link
Copy Markdown
Member

Previously, GitHub info sync is only triggerred by /bump or /openpr calls, so user rows of new contributors are created only when they use /bump or /openpr.

However other commands, such as /pr and /token, may also require user row, so run GitHub info sync immediately upon login to insert that.

Previously, GitHub info sync is only triggerred
by /bump or /openpr calls, so user rows of new
contributors are created only when they use
/bump or /openpr.

However other commands, such as /pr and /token,
may also require user row, so run GitHub info sync
immediately upon login to insert that.

Signed-off-by: xtex <xtex@astrafall.org>

@MingcongBai MingcongBai left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested good in production.

@MingcongBai
MingcongBai merged commit f18e49a into master Apr 6, 2026
1 check passed
@xtexx
xtexx deleted the login-fix branch April 11, 2026 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants