Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🔀 Dev to prod #820

Closed
wants to merge 11 commits into from
Closed

🔀 Dev to prod #820

wants to merge 11 commits into from

Conversation

Mario-SO
Copy link
Member

No description provided.

Mario-SO and others added 11 commits July 25, 2024 07:04
Refactor ImageUpload component for banner and profile image handling- 🔨
Enhanced ImageUpload component to handle both banner and profile image
uploads

- ✨ Implemented client-side image dimension validation (1500x500px for
banner)
- 💬 Added error messaging for invalid image sizes

- 🎨 Improved styling for profile image upload:
- 🔄 Rounded placeholder for profile image
- 🔧 Correct positioning relative to banner

- 🚸 Adjusted error message display to appear below upload area
- 🔇 Removed placeholder text for profile image upload
- 🏷️ Updated component to use TypeScript for improved type safety
- 🔗 Ensured compatibility with react-hook-form in CreateOrganizationForm
- 📱 Optimized layout and styling for better user experience
- 🆕 Added ConfirmImageDeletion dialog for image removal

This refactor improves the flexibility and usability of the ImageUpload
component, allowing it to be used for different image upload scenarios
while maintaining a consistent user interface and experience.
I will need to send this to our lawyer to double check and change the
email of contact.

- [x] Add new `data-request` page
…ze-on-settings-page

🐛 Fixes #728 + some more enhancements
Copy link

vercel bot commented Jul 26, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
streameth-platform ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 26, 2024 9:21am

@Mario-SO Mario-SO closed this Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants