Skip to content

Replace rocket tag with acts_as_taggable_on [#480] #279

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

Merged
merged 6 commits into from
Jan 2, 2015

Conversation

rohitpaulk
Copy link
Contributor

Bounty#480

This PR also restores the ProTip specs.

When deploying, run bundle install before migrating the database.

@rohitpaulk rohitpaulk force-pushed the replace-rocket-tag-with-aato branch from 2dd7c8b to 3dd27c1 Compare January 1, 2015 06:00
@seuros
Copy link
Contributor

seuros commented Jan 1, 2015

Awesome!

Could you remove 636f915 ? It part of another PR .

There were a few leftovers from rocket_tag in the networks, opportunities and protips controllers. + indentation fix on migration
@rohitpaulk
Copy link
Contributor Author

@seuros - I fixed the tests as they tested a lot of the functionality that is modified by this PR. What exactly do you want me to do? Pick out that commit and open another PR?

@rohitpaulk rohitpaulk force-pushed the replace-rocket-tag-with-aato branch from 3dd27c1 to e991348 Compare January 1, 2015 09:03
Space is no longer used as a delimiter anywhere in the codebase. Updated specs. minor fixes
@rohitpaulk rohitpaulk force-pushed the replace-rocket-tag-with-aato branch from e991348 to ac2cb88 Compare January 1, 2015 15:37
mdeiters added a commit that referenced this pull request Jan 2, 2015
Replace rocket tag with acts_as_taggable_on [#480]
@mdeiters mdeiters merged commit 92e5d6a into coderwall:master Jan 2, 2015
rohitpaulk added a commit to rohitpaulk/coderwall that referenced this pull request Jan 10, 2015
This also fixes a small issue that was leftover from coderwall#279
rohitpaulk added a commit to rohitpaulk/coderwall that referenced this pull request Jan 10, 2015
This also fixes a small issue that was leftover from coderwall#279
rohitpaulk added a commit to rohitpaulk/coderwall that referenced this pull request Jan 10, 2015
Regression from coderwall#279
rohitpaulk added a commit to rohitpaulk/coderwall that referenced this pull request Jan 11, 2015
Reorganize Plan model, Remove duplicated uniqueness validations, Fix seeds.rb

Regression from coderwall#279, Remove payer_is_team_admin
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.

3 participants