Skip to content

Enhance authentication error handling in AuthForm and related components#372

Merged
Stivenjs merged 7 commits into
mainfrom
cli
Oct 27, 2025
Merged

Enhance authentication error handling in AuthForm and related components#372
Stivenjs merged 7 commits into
mainfrom
cli

Conversation

@Stivenjs
Copy link
Copy Markdown
Contributor

This commit introduces improved error handling for Google authentication in the AuthForm component. It adds a state to manage Google-specific error messages and integrates the getSignInErrorMessage function to provide user-friendly feedback based on authentication errors. Additionally, it refactors the form rendering logic to ensure that error messages are displayed appropriately across different authentication states, enhancing the overall user experience during sign-in and sign-up processes.

This commit introduces improved error handling for Google authentication in the AuthForm component. It adds a state to manage Google-specific error messages and integrates the getSignInErrorMessage function to provide user-friendly feedback based on authentication errors. Additionally, it refactors the form rendering logic to ensure that error messages are displayed appropriately across different authentication states, enhancing the overall user experience during sign-in and sign-up processes.
…dling

This commit refactors the StorePage component by replacing direct logic with the StorePageController and StoreMetadataController for better separation of concerns. It removes unused imports and enhances error handling to render HTML error messages when available. Additionally, it updates the generateMetadata function to utilize the new controller, streamlining metadata generation. The DevAutoReloadScript and store-page-utils have been removed as they are no longer needed, simplifying the codebase.
This commit refactors the useOrderNotifications hook by updating the import paths for getOrderStatus and getPaymentStatus functions, enhancing clarity and maintainability. The changes streamline the notification handling logic related to orders, ensuring better organization of the codebase.
This commit updates the import paths in several files for improved clarity and organization. It modifies the import statement in `next-env.d.ts` to reference the correct routes type definition, consolidates imports in `verify-controller.ts` for better readability, and refines the import structure in `useOrderNotifications.ts` and `order-fetcher.ts` to utilize server-specific notification services. Additionally, it adds a note in `index.ts` to clarify the server-only nature of certain exports, ensuring a cleaner and more maintainable codebase.
…r import conversion

This commit updates various import paths across the liquid-forge package to use relative references instead of absolute paths, enhancing clarity and maintainability. Additionally, it introduces new scripts in the package.json for converting imports to relative paths, facilitating easier refactoring in the future. The pnpm-lock.yaml file is also updated to include the tsx package for running the conversion scripts.
…tainability

This commit updates various filters and tags within the liquid-forge package to replace hardcoded regex patterns with centralized pattern definitions from the regex-patterns module. This change enhances code readability and maintainability by consolidating pattern management. Additionally, it modifies the import paths in several files for better organization and clarity, ensuring a more streamlined codebase.
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Oct 27, 2025
Comment thread packages/liquid-forge/services/templates/parsing/schema-parser.ts Dismissed
Comment thread packages/liquid-forge/services/templates/parsing/schema-parser.ts Dismissed
Comment thread packages/liquid-forge/services/themes/optimization/postcss-processor.ts Dismissed
Comment thread packages/liquid-forge/services/themes/optimization/postcss-processor.ts Dismissed
Comment thread packages/liquid-forge/services/themes/optimization/postcss-processor.ts Dismissed
Comment thread packages/liquid-forge/services/themes/validation/rules/performance-rules.ts Dismissed
Comment thread packages/liquid-forge/services/themes/validation/rules/performance-rules.ts Dismissed
Comment thread packages/liquid-forge/services/themes/validation/rules/security-rules.ts Dismissed
Comment thread packages/liquid-forge/services/themes/validation/rules/security-rules.ts Dismissed
@Stivenjs Stivenjs merged commit 3dd2f6c into main Oct 27, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants