Skip to content
This repository has been archived by the owner on Mar 15, 2018. It is now read-only.

remove preload test plan feature (bug 1164223) #3433

Merged
merged 1 commit into from
Nov 11, 2015

Conversation

ngokevin
Copy link
Contributor

@ngokevin ngokevin commented Nov 9, 2015

No description provided.

"note": "Submission process for preinstalled apps."
},
"model": "waffle.switch",
"pk": 48
Copy link
Member

Choose a reason for hiding this comment

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

Consider a migration to remove the waffle switch as well, maybe ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i added one, not sure if i did it right though

Copy link
Member

Choose a reason for hiding this comment

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

Looks good

@diox
Copy link
Member

diox commented Nov 9, 2015

travis found a bunch of orphaned imports

@ngokevin ngokevin force-pushed the rmpreload branch 3 times, most recently from 398cecc to f77a52e Compare November 9, 2015 21:46


def remove_preload_waffle(apps, schema_editor):
# We can't import the Person model directly as it may be a newer
Copy link
Member

Choose a reason for hiding this comment

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

s/Person/Switch/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

got caught red-handed copying from the docs

@ngokevin
Copy link
Contributor Author

@diox re-r? added a migration to wipe the table, don't need it

@diox
Copy link
Member

diox commented Nov 11, 2015

r+

ngokevin added a commit that referenced this pull request Nov 11, 2015
remove preload test plan feature (bug 1164223)
@ngokevin ngokevin merged commit abec2d5 into mozilla:master Nov 11, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants