Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

"in depth" -> "in-depth" #1190

Closed
wants to merge 1 commit into from
Closed

"in depth" -> "in-depth" #1190

wants to merge 1 commit into from

Conversation

brettcannon
Copy link
Contributor

No description provided.

@btford
Copy link
Contributor

btford commented Jul 21, 2012

Merged. Thanks!

@btford btford closed this Jul 21, 2012
IgorMinar pushed a commit to IgorMinar/angular.js that referenced this pull request Feb 7, 2013
Safari and IE don't like being told to store cookies with path set to
undefined. This change ensures that if base[href] (from which cookie path
is derived) is undefined then the cookie path defaults to ''.

The test verifies that the cookie is set instead of checking that cookie has correct path,
this is due to that cookie meta information is not avabile once the cookie is set.

Closes angular#1190, angular#1191
IgorMinar pushed a commit that referenced this pull request Feb 7, 2013
Safari and IE don't like being told to store cookies with path set to
undefined. This change ensures that if base[href] (from which cookie path
is derived) is undefined then the cookie path defaults to ''.

The test verifies that the cookie is set instead of checking that cookie has correct path,
this is due to that cookie meta information is not avabile once the cookie is set.

Closes #1190, #1191
IgorMinar pushed a commit that referenced this pull request Feb 7, 2013
Safari and IE don't like being told to store cookies with path set to
undefined. This change ensures that if base[href] (from which cookie path
is derived) is undefined then the cookie path defaults to ''.

The test verifies that the cookie is set instead of checking that cookie has correct path,
this is due to that cookie meta information is not avabile once the cookie is set.

Closes #1190, #1191
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