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

fix: Migrate to Rails 5.2 #92

Open
wants to merge 82 commits into
base: master
Choose a base branch
from
Open

fix: Migrate to Rails 5.2 #92

wants to merge 82 commits into from

Conversation

azizmashkour
Copy link

No description provided.

wazery and others added 30 commits July 6, 2014 16:25
Conflicts:
	README.md
	lib/generators/ratyrate/templates/ratyrate.js.erb
	lib/ratyrate/helpers.rb
add db:migrate to README, fix inconsistent capitalization
fix stars not display with turbolinks
Small fixes to make STI working correctly
Fix error when reflecting on associations.
data-cancel should be treated as boolean value, not string
Nicolas and others added 30 commits May 14, 2015 08:28
Fix jquery.raty.js not found error
Change image path for gem's view helpers
Changes on lines 727,728, 748,749 and 750 
Original 
 cancelOff    : '<%= asset_path('cancel-off.png') %>',
 cancelOn     : '<%= asset_path('cancel-on.png') %>',

Changed
  cancelOff    : '<%= asset_path("cancel-off.png") %>',
  cancelOn     : '<%= asset_path("cancel-on.png") %>',
Trigger an event (rated) when there is a new rating

Docs needs to be updated later on
Also removes CoderWall endorsement counter
Updated jquery-rails version to `>= 4.3.1`
Fix rate_for_user recognizes wrong rateable_type for polymorphic
* Added migration version to handle 'Directly inheriting from ActiveRecord::Migration is not supported', this error

* removed unused else condition when rails version is less than 5
for migration_version
fix: Migrate to Rails 5.2 and fix all migration conflicts
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.