Fix deploy #33
Closed
Fix deploy #33
Conversation
This file contains 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
…environment - Added radix-utils polyfill that ensures React.useLayoutEffect is available - Added polyfill import to all Radix UI components - Prevents 'Cannot read properties of undefined (reading useLayoutEffect)' error
…source - Temporarily replaced all @radix-ui components with simple React mockups - Fixed TypeScript errors in UserDataModal - This is a diagnostic commit to identify which component causes the error
…Effect - Added comprehensive React polyfill that handles createContext, useLayoutEffect, etc. - Added the polyfill to index.html as a script tag to ensure it runs before any React code - Reverted mock components back to original Radix UI components
…ntext errors - Added react-shim.ts that polyfills all required React global methods - Added import statements to all Radix UI component files - Updated index.html to include more robust React polyfill with non-configurable properties - Improved TypeScript type handling for global React object
- Added SimpleDialog, SimpleTooltip, and SimpleLabel components as pure React alternatives - Created a compatibility layer in radix-compatibility.tsx to minimize code changes - Updated UserDataModal and App to use the new components - This is a first step toward removing Radix UI dependency completely
- Removed all Radix UI dependencies from package.json - Created simple replacements for Dialog, Tooltip, Dropdown, Label, Slot, etc. - Created a compatibility layer to minimize code changes - Updated all component imports to use the new non-Radix components - This should fix the useLayoutEffect and createContext errors in production
- Fixed 'Cannot assign to read only property React of object #<Window>' error - Removed problematic polyfill from index.html that was causing conflicts - Improved the React shim to safely check properties before setting them - Added better error handling to quietly handle read-only property issues
- Created a new better-tooltip component with proper hover and positioning - Uses React context and portals for best implementation - Replaces the dummy tooltip placeholders with actual working components - Maintains API compatibility with Radix UI tooltips
AlexVOiceover
temporarily deployed
to
production
GitHub Actions
Inactive
AlexVOiceover
temporarily deployed
to
production
GitHub Actions
Inactive
AlexVOiceover
temporarily deployed
to
production
GitHub Actions
Inactive
AlexVOiceover
temporarily deployed
to
production
GitHub Actions
Inactive
AlexVOiceover
temporarily deployed
to
production
GitHub Actions
Inactive
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.