Skip to content

Commit caa4d79

Browse files
authored
Merge pull request #2580 from appwrite/feat-move-github-button-above
2 parents 0240897 + 0a8d7ee commit caa4d79

2 files changed

Lines changed: 28 additions & 20 deletions

File tree

src/routes/(public)/(guest)/login/+page.svelte

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,15 @@
9494
<svelte:fragment>
9595
<Form onSubmit={login}>
9696
<Layout.Stack>
97+
{#if isCloud}
98+
<div style:margin-bottom="var(--gap-s, 8px)">
99+
<Button secondary fullWidth on:click={onGithubLogin} {disabled}>
100+
<span class="icon-github" aria-hidden="true"></span>
101+
<span class="text">Sign in with GitHub</span>
102+
</Button>
103+
</div>
104+
<span class="with-separators eyebrow-heading-3">or</span>
105+
{/if}
97106
<InputEmail
98107
id="email"
99108
label="Email"
@@ -108,13 +117,6 @@
108117
required={true}
109118
bind:value={pass} />
110119
<Button fullWidth submit {disabled}>Sign in</Button>
111-
{#if isCloud}
112-
<span class="with-separators eyebrow-heading-3">or</span>
113-
<Button secondary fullWidth on:click={onGithubLogin} {disabled}>
114-
<span class="icon-github" aria-hidden="true"></span>
115-
<span class="text">Sign in with GitHub</span>
116-
</Button>
117-
{/if}
118120
</Layout.Stack>
119121
</Form>
120122
</svelte:fragment>

src/routes/(public)/(guest)/register/+page.svelte

Lines changed: 19 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -106,9 +106,17 @@
106106
}
107107
108108
function onGithubLogin() {
109+
let successUrl = window.location.origin;
110+
111+
if (page.url.searchParams.has('code')) {
112+
successUrl += `?code=${page.url.searchParams.get('code')}`;
113+
} else if (page.url.searchParams.has('campaign')) {
114+
successUrl += `?campaign=${page.url.searchParams.get('campaign')}`;
115+
}
116+
109117
sdk.forConsole.account.createOAuth2Session({
110118
provider: OAuthProvider.Github,
111-
success: window.location.origin,
119+
success: successUrl,
112120
failure: window.location.origin,
113121
scopes: ['read:user', 'user:email']
114122
});
@@ -124,6 +132,16 @@
124132
<svelte:fragment>
125133
<Form onSubmit={register}>
126134
<Layout.Stack>
135+
{#if isCloud}
136+
<div style:margin-bottom="var(--gap-s, 8px)">
137+
<Button secondary fullWidth on:click={onGithubLogin} {disabled}>
138+
<span class="icon-github" aria-hidden="true"></span>
139+
<span class="text">Sign up with GitHub</span>
140+
</Button>
141+
</div>
142+
<span class="with-separators eyebrow-heading-3">or</span>
143+
{/if}
144+
127145
<InputText
128146
id="name"
129147
label="Name"
@@ -159,18 +177,6 @@
159177
>.</InputChoice>
160178

161179
<Button fullWidth submit disabled={disabled || !terms}>Sign up</Button>
162-
163-
{#if isCloud}
164-
<span class="with-separators eyebrow-heading-3">or</span>
165-
<Button
166-
secondary
167-
fullWidth
168-
on:click={onGithubLogin}
169-
disabled={disabled || !terms}>
170-
<span class="icon-github" aria-hidden="true"></span>
171-
<span class="text">Sign up with GitHub</span>
172-
</Button>
173-
{/if}
174180
</Layout.Stack>
175181
</Form>
176182
</svelte:fragment>

0 commit comments

Comments
 (0)