-
Notifications
You must be signed in to change notification settings - Fork 29
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
mark elephpants as prototypes #174
base: master
Are you sure you want to change the base?
mark elephpants as prototypes #174
Conversation
I think the Vegas Elephpant didn't reached the kickstarter goal, or? Will test your PR at the end of the week when everything works out. |
Yeah pretty much, it was done on Indiegogo which is pretty much the same thing. It only reached 27% of the target so the prototype is the only one that exists 😄 https://www.indiegogo.com/projects/phplashy-the-official-php-vegas-elephpant#/ However, on the statistics page, it's apparently in 2 herds: I'll tidy up this branch this evening and resolve any conflicts, ready for review later this week. Thanks |
This one is real |
Such a shame it didn't get the funding it needed 😢 |
Hey @teiling88 👋 I've resolved the conflicts on this branch so it's ready whenever you are for a test :) |
Hey @teiling88 👋 Activity on this repo seems to have dried up a little, so I'd like to kickstart it! Have you got some time to give this PR a road test? The Kickstarter page for the Sylius Elephpant is launching soon and that's what made me remember about this PR! Thanks |
I just realised this branch had a merge conflict, I've sorted this now. |
70ec0f5
to
9cbcebb
Compare
Pushed an update which removes the Prototype flag from the Sylius Elephpant as it reached its funding goal yesterday so it can go into production. |
Might be a bit premature since manufacturing hasn't even started |
a75934e
to
9cbcebb
Compare
Yeah I've reverted that commit, I guess the excitement of a new Elephpant got the better of me 😆 |
The original batch of one of the Symonfy elePHPants (I don't remember which one) was shipped from the factory to France. French customs didn't like them and disposed of them the way they do with anything else they don't want to let in the country: the incinerator. |
Poor Elephpants! I didn't know that happened! Thanks for the info! |
Fixes #126
Following on from a discussion started by @asgrim in #126, this PR adds the ability to be able to mark Elephpants as "prototypes", which will then prevent them from being able to be added to collections and offered up for trade. This will mark the following as Prototypes:
Prototype Elephpants will only show on the following pages:
Prototype Elephpants will be excluded from:
Lastly, I've added an extra migration to remove any Prototype Elephpants from collections.