Misc changes to docker imaging and other various bug fixes #15
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…onment setup - Added important notes for NEXTAUTH_SECRET and NEXTAUTH_URL in env.example to clarify their roles in authentication. - Updated next.config.ts to conditionally enable standalone output for production builds. - Included a postinstall script in package.json to handle potential Prisma generate issues on Windows. - Enhanced error handling in the NextAuth route to return JSON errors instead of HTML. - Updated README.md with a note for Windows users regarding symlink permission errors during Prisma generation. - Removed unused scanning UI elements in BarcodeScanner component for cleaner code.
- Updated the wrappedHandler function to await async parameters in Next.js 16+, ensuring proper handling of request context. - Enhanced error logging for better debugging of authentication route issues.
- Added a new line at the end of various files including scripts, markdown documentation, and component files to ensure consistency and adhere to coding standards. - This change improves readability and prevents potential issues with file concatenation in certain environments.
- Added new environment files (.env.docker, .env.cli) to support separate configurations for Docker and CLI development. - Updated docker-compose.yml to remove container names and improve volume management for certificates. - Enhanced README.md and DOCKER.md with instructions for running Docker Compose without interfering with local development. - Introduced new npm scripts for managing Docker containers easily.
…ositioning - Updated the inventory deletion logic to provide more specific error messages based on server responses, including session expiration notifications. - Improved the dropdown menu component by adding a trigger reference and implementing dynamic positioning to ensure it displays correctly within the viewport, using a portal for rendering.
- Excluded both server.js and prisma.config.js from TypeScript rules in ESLint configuration for better compatibility. - Refactored AdminUsersClient to use useCallback for fetchUsers, improving performance by preventing unnecessary re-creations of the function. - Updated RecipeDetailPage and other API routes to remove unused parameters and mark them as intentionally unused for future reference. - Cleaned up BarcodeScanner component by removing unnecessary state management for scanning, streamlining the code. - Added comments to clarify the purpose of unused variables in various API routes, ensuring better maintainability.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.