Skip to content

Dropping the database invalidates cookies#504

Merged
ErisDS merged 1 commit intoTryGhost:masterfrom
javorszky:iss418
Aug 24, 2013
Merged

Dropping the database invalidates cookies#504
ErisDS merged 1 commit intoTryGhost:masterfrom
javorszky:iss418

Conversation

@javorszky
Copy link
Contributor

Closes #418.

  • Moved the app config behind the promise wall (I couldn't reliably assign generated uuid to the ghost object AND have access to an automatically created db from fixtures AND not have circular reference (try including api in ghost.js ;) ))
  • Added new functionality to ghost.init(), which is responsible for the first run bit (I'm thinking plopping a filter or an action in there for future devs)
  • Modified .gitignore so the .pngs casper generates aren't added
  • Fixed ambiguity and typos here and there, see code

Closes #418.
* Moved the app config behind the promise wall (I couldn't reliably assign generated uuid to the ghost object AND have access to an automatically created db from fixtures AND not have circular reference (try including api in ghost.js ;) ))
* Added new functionality to `ghost.init()`, which is responsible for the first run bit (I'm thinking plopping a filter or an action in there for future devs)
* Modified `.gitignore` so the `.png`s casper generates aren't added
* Fixed ambiguity and typos here and there, see code
@ErisDS
Copy link
Member

ErisDS commented Aug 24, 2013

Had a look over this, looks good 👍 want to give it a bit of a test before merging tho. Hang tight :)

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.

Login session does not invalidate with new DB

2 participants