Skip to content

Commit

Permalink
Merge pull request #98 from CriticalMoments/misspell
Browse files Browse the repository at this point in the history
Add `Misspell` tool to detect typos, and fix all found typos
  • Loading branch information
scosman authored Jul 20, 2024
2 parents 2026a8d + cceceb3 commit 27fe583
Show file tree
Hide file tree
Showing 8 changed files with 14 additions and 7 deletions.
7 changes: 7 additions & 0 deletions .github/workflows/format.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,16 @@ jobs:

steps:
- uses: actions/checkout@v4
- uses: actions/setup-go@v5

- name: NPM install
run: npm install

- name: Check Code Format
run: npm run format_check

- name: Download go misspell utility
run: go install github.com/client9/misspell/cmd/misspell@latest

- name: Run misspell to fine misspelled words
run: misspell -error ./src README.md
2 changes: 1 addition & 1 deletion src/routes/(admin)/account/(menu)/billing/+page.svelte
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@
initialValue: currentPlanName || "",
},
]}
editButtonTitle="Manage Subscripton"
editButtonTitle="Manage Subscription"
editLink="/account/billing/manage"
/>
{/if}
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
title="Change Email"
editable={true}
successTitle="Email change initiated"
successBody="You should recieve emails at both the old and new address to confirm the change. Please click the link in both emails to finalized the change. Until finalized, you must sign in with your current email."
successBody="You should receive emails at both the old and new address to confirm the change. Please click the link in both emails to finalized the change. Until finalized, you must sign in with your current email."
formTarget="/account/api?/updateEmail"
fields={[
{
Expand Down
2 changes: 1 addition & 1 deletion src/routes/(admin)/account/api/+page.server.ts
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ export const actions = {
}

// Check current password is correct before updating, but only if they didn't log in with "recover" link
// Note: to make this truely enforced you need to contact supabase. See: https://www.reddit.com/r/Supabase/comments/12iw7o1/updating_password_in_supabase_seems_insecure/
// Note: to make this truly enforced you need to contact supabase. See: https://www.reddit.com/r/Supabase/comments/12iw7o1/updating_password_in_supabase_seems_insecure/
// However, having the UI accessible route still verify password is still helpful, and needed once you get the setting above enabled
if (!isRecoverySession) {
const { error } = await supabase.auth.signInWithPassword({
Expand Down
2 changes: 1 addition & 1 deletion src/routes/(marketing)/+page.svelte
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@
<path d="M7 16H17" stroke="#1C274C" stroke-width="1.5" stroke-linecap="round"/>`,
},
{
name: "Scaleable",
name: "Scalable",
description:
"Handle new users and scale with horizontally scaling edge functions.",
svgContent: `<path d="M3 22H21" stroke="#1C274C" stroke-width="1.5" stroke-linecap="round" stroke-linejoin="round"/>
Expand Down
2 changes: 1 addition & 1 deletion src/routes/(marketing)/login/sign_in/+page.svelte
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
onMount(() => {
supabase.auth.onAuthStateChange((event) => {
// Redirect to account after sucessful login
// Redirect to account after successful login
if (event == "SIGNED_IN") {
// Delay needed because order of callback not guaranteed.
// Give the layout callback priority to update state or
Expand Down
2 changes: 1 addition & 1 deletion src/routes/(marketing)/pricing/pricing_plans.ts
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ export const pricingPlans = [
id: "enterprise",
name: "Enterprise",
description:
"A plan to test the upgrade expereince. Try buying this with the test credit card 4242424242424242.",
"A plan to test the upgrade experience. Try buying this with the test credit card 4242424242424242.",
price: "$15",
priceIntervalName: "per month",
stripe_price_id: "price_1Nkda2HMjzZ8mGZn4sKvbDAV",
Expand Down
2 changes: 1 addition & 1 deletion src/routes/+error.svelte
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
<div class="hero min-h-[100vh]">
<div class="hero-content text-center">
<div class="max-w-lg">
<h1 class="text-5xl font-bold">This is embarassing...</h1>
<h1 class="text-5xl font-bold">This is embarrassing...</h1>
<p class="py-6 text-2xl">There was an error: {$page?.error?.message}</p>
<div>
<a href="/" class="btn btn-primary btn-wide">Return Home</a>
Expand Down

0 comments on commit 27fe583

Please sign in to comment.