Bug #4718

Site's tagging library should allow initial migrations to run

Added by Igal Koshevoy over 1 year ago. Updated 2 months ago.

Status:Closed Start date:09/04/2010
Priority:High Due date:
Assignee:- % Done:

0%
Category:module site Spent time: -
Target version:-
Keywords: Affected URL:
Branch:https://github.com/nicklewis/puppet-forge/tree/ticket/master/4718
Votes: 0

Description

Currently if you checkout the code and run rake db:create and then rake db:migrate, it’ll fail because the acts-as-taggable-on tagging library demands that tables exist even before they’re created.

There are a number of ugly workarounds available. There is also a new version of the library available that addresses this problem … but changes the API and data model. Need to figure out what to do.

This is a low priority problem because it makes setting up a database difficult for those few developing the product, and they can bypass the problem by getting a copy of the production database from me so they can move past this bug.

Related issues

related to Puppet Forge - Bug #11856: Getting the forge setup and passing tests should be docum... In Topic Branch Pending Review 01/09/2012

History

#1
Updated by Igal Koshevoy over 1 year ago

  • Assignee deleted (Igal Koshevoy)

Unassigning because this was less urgent and I didn’t end up working on it.

#2
Updated by Igal Koshevoy over 1 year ago

  • Priority changed from Low to High

Upgrading status to “High” because we want more people to work on Forge and it’s impossible to run the migrations currently.

#3
Updated by Anonymous over 1 year ago

  • Target version changed from 55 to 73

#4
Updated by Nick Lewis 6 months ago

  • Status changed from Accepted to In Topic Branch Pending Review
  • Affected URL set to https://github.com/nicklewis/puppet-forge/tree/ticket/master/4718

Pull request 2 addresses this, upgrading acts-as-taggable-on to version 2.1.1. It also has a few adjustments to our other gem dependencies, to make it possible to use the application after running the gems:install task.

#5
Updated by Nick Lewis 6 months ago

  • Branch set to https://github.com/nicklewis/puppet-forge/tree/ticket/master/4718
  • Affected URL deleted (https://github.com/nicklewis/puppet-forge/tree/ticket/master/4718)

#6
Updated by Ben Hughes 5 months ago

  • Status changed from In Topic Branch Pending Review to Closed

Deployed today – 2012/01/04.

#7
Updated by Daniel Pittman 2 months ago

  • Target version deleted (73)

Also available in: Atom PDF