fix (dashboard): fix password reset redirect url (#3391)
### **PR Type** Bug fix, Enhancement ___ ### **Description** - Fixed password reset redirect to use absolute URL. - Removed duplicate sign-up tabs and extra legal text. - Added direct sign-in prompt for existing users. - Added changeset entry for the patch update. ___ ### **Changes diagram** ```mermaid flowchart LR A["Password reset redirect (relative)"] -- "Changed to absolute URL" --> B["Password reset redirect (absolute)"] C["Sign-up page with tabs and legal text"] -- "Removed tabs & extra legal text" --> D["Sign-up page with sign-in prompt"] E["No changeset for fix"] -- "Added changeset entry" --> F["Patch changeset for dashboard"] ``` ___ ### **Changes walkthrough** 📝 <table><thead><tr><th></th><th align="left">Relevant files</th></tr></thead><tbody><tr><td><strong>Bug fix</strong></td><td><table> <tr> <td> <details> <summary><strong>new.tsx</strong><dd><code>Use absolute URL for password reset redirect</code> </dd></summary> <hr> dashboard/src/pages/password/new.tsx <li>Changed password reset redirect to use absolute URL.<br> <li> Ensures correct redirect after password reset email. </details> </td> <td><a href="https://github.com/nhost/nhost/pull/3391/files#diff-153045bbcb44ce952fbd9ee585c63109891973ab4d1ecc1e1b5edf8f981b1259">+1/-1</a> </td> </tr> </table></td></tr><tr><td><strong>Enhancement</strong></td><td><table> <tr> <td> <details> <summary><strong>signup.tsx</strong><dd><code>Streamlined sign-up page and added sign-in prompt</code> </dd></summary> <hr> dashboard/src/pages/signup.tsx <li>Removed <code>SignUpTabs</code> and duplicate legal text.<br> <li> Added prompt for existing users to sign in.<br> <li> Simplified and clarified sign-up page UI. </details> </td> <td><a href="https://github.com/nhost/nhost/pull/3391/files#diff-fc2b5989e3bbafda1d3d8b2317d24c39ef2b8cec0c4dc410170fa2da13464f68">+4/-19</a> </td> </tr> </table></td></tr><tr><td><strong>Documentation</strong></td><td><table> <tr> <td> <details> <summary><strong>serious-eagles-rule.md</strong><dd><code>Added changeset for password reset redirect fix</code> </dd></summary> <hr> .changeset/serious-eagles-rule.md <li>Added changeset entry for password reset redirect fix.<br> <li> Documents patch update for dashboard package. </details> </td> <td><a href="https://github.com/nhost/nhost/pull/3391/files#diff-058395bb4a26b44e46054945b2710688aebefcb9ad898396af1cac658e20b049">+5/-0</a> </td> </tr> </table></td></tr></tr></tbody></table> ___ > <details> <summary> Need help?</summary><li>Type <code>/help how to ...</code> in the comments thread for any questions about PR-Agent usage.</li><li>Check out the <a href="https://qodo-merge-docs.qodo.ai/usage-guide/">documentation</a> for more information.</li></details>
This commit is contained in:
5
.changeset/serious-eagles-rule.md
Normal file
5
.changeset/serious-eagles-rule.md
Normal file
@@ -0,0 +1,5 @@
|
||||
---
|
||||
'@nhost/dashboard': patch
|
||||
---
|
||||
|
||||
fix (dashboard): fix password reset redirect url
|
||||
9
.github/workflows/ci.yaml
vendored
9
.github/workflows/ci.yaml
vendored
@@ -172,10 +172,11 @@ jobs:
|
||||
- name: Set Dashboard Preview URL
|
||||
if: steps.fetch-dashboard-preview-url.outputs.preview_url != ''
|
||||
run: echo "NHOST_TEST_DASHBOARD_URL=https://${{ steps.fetch-dashboard-preview-url.outputs.preview_url }}" >> $GITHUB_ENV
|
||||
- name: Run Upgrade project Dashboard e2e tests
|
||||
if: matrix.package.path == 'dashboard'
|
||||
timeout-minutes: 10
|
||||
run: pnpm --filter="${{ matrix.package.name }}" run e2e:upgrade-project
|
||||
# Turning it off temporarly
|
||||
# - name: Run Upgrade project Dashboard e2e tests
|
||||
# if: matrix.package.path == 'dashboard'
|
||||
# timeout-minutes: 10
|
||||
# run: pnpm --filter="${{ matrix.package.name }}" run e2e:upgrade-project
|
||||
# * Run the `ci` script of the current package of the matrix. Dependencies build is cached by Turborepo
|
||||
- name: Run e2e tests
|
||||
timeout-minutes: 20
|
||||
|
||||
@@ -46,7 +46,7 @@ export default function NewPasswordPage() {
|
||||
await nhost.auth.sendPasswordResetEmail({
|
||||
email,
|
||||
options: {
|
||||
redirectTo: '/password/reset',
|
||||
redirectTo: `${window.location.origin}/password/reset`,
|
||||
},
|
||||
});
|
||||
setIsSent(true);
|
||||
|
||||
@@ -86,25 +86,10 @@ export default function SignUpPage() {
|
||||
</NextLink>
|
||||
</p>
|
||||
</div>
|
||||
<SignUpTabs />
|
||||
<Divider className="!my-2" />
|
||||
<p className="text-center text-sm text-[#A2B3BE]">
|
||||
By signing up, you agree to our{' '}
|
||||
<NextLink
|
||||
href="https://nhost.io/legal/terms-of-service"
|
||||
target="_blank"
|
||||
rel="noopener noreferrer"
|
||||
className="font-semibold text-white"
|
||||
>
|
||||
Terms of Service
|
||||
</NextLink>{' '}
|
||||
and{' '}
|
||||
<NextLink
|
||||
href="https://nhost.io/legal/privacy-policy"
|
||||
rel="noopener noreferrer"
|
||||
className="font-semibold text-white"
|
||||
>
|
||||
Privacy Policy
|
||||
<p className="text-center text-base text-[#A2B3BE] lg:text-lg">
|
||||
Already have an account?{' '}
|
||||
<NextLink href="/signin" className="font-medium text-white">
|
||||
Sign In
|
||||
</NextLink>
|
||||
</p>
|
||||
</div>
|
||||
|
||||
Reference in New Issue
Block a user