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

Add support to Rails 7.1 #74

Merged
merged 3 commits into from
Sep 19, 2023
Merged

Add support to Rails 7.1 #74

merged 3 commits into from
Sep 19, 2023

Conversation

santib
Copy link
Member

@santib santib commented Jun 7, 2023

Summary

  • Updated Gemfile for Rails 7.1 support
    The Gemfile has been updated to ensure compatibility with Rails 7.1, keeping the project up-to-date with the latest Rails version.

  • New test added for blank base64 data
    A new test has been added to the user_base64_spec to ensure that blank base64 data is properly handled by the system.

  • Bug fix for assigning pictures with nil or empty string
    A bug related to handling picture assignments via the setter method has been fixed. The issue occurred when the input was either nil or an empty string, and it now correctly handles these cases.

@@ -15,7 +15,7 @@ Gem::Specification.new do |s|
s.required_ruby_version = '>= 2.7.0'

# Dependencies
s.add_dependency 'rails', '>= 7.0'
s.add_dependency 'rails', '>= 7.1'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you accept the new Rails version 7.1.0.beta1 please ? :)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cprodhomme done, version 3.0.0 released. Let me know if it works for you. Thanks

@santib santib force-pushed the fix-rails-head branch 2 times, most recently from c04a79a to 205f5bc Compare September 19, 2023 15:07
@santib santib merged commit fd5c4e7 into master Sep 19, 2023
16 checks passed
@santib santib deleted the fix-rails-head branch September 19, 2023 18:03
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.

4 participants