-
Notifications
You must be signed in to change notification settings - Fork 213
Include note about making sure cookies are enabled for the herokuapp domain #3
base: master
Are you sure you want to change the base?
Conversation
|
wow, that's a real corner case ... how many people will have cookies disabled for that domain? Is it worth adding this if only 1 in a thousand will encounter it @freeranger ? |
|
@tansaku Granted there may not be that many people who are so security conscious, but given that it can happen, why not mention it in the notes? |
|
because for everyone who doesn't have to deal with it then it's an extra thing for them to read that makes the homework more daunting. I think I'd be more comfortable with a footnote - or a troubleshooting section that mentioned it to avoid the main description become too bloated ... |
|
Fair point, though if they don't understand about cookies, what are they doing on a web course (part 2)? :) Maybe a separate "troubleshooting" file in the repo...but how do they get sight of that easily? |
|
yeah I think a separate troubleshooting file, and we link to that at beginning and end; however I don't think that we need to worry too much about that being in site. I'm not saying they won't understand cookies - just that we want to avoid bloat on items that will only benefit 1 in a 1000. Of course if I'm wrong and 90% of folks encounter this that's another thing. However assuming this is for 1 in a 1000 thing I think we should avoid verbiage for the other 999 who will appreciate brevity wherever possible I think ... |
|
Ok I'll redo it in a separate file - it will be a bit lonely though coz I don't have any other troubleshooting tips to share ;) Do you want to reject the pull request and I will do another or what?
|
|
@freeranger you could re-use this pull request if you liked by dumping the old commit and starting a new one - that way we keep all the related discussion? |
|
@freeranger - were you going to add to this PR? If you want to open another that's fine - I'll close and work from the other |
Student had a problem with cookies disabled:
https://courses.edx.org/courses/course-v1:BerkeleyX+CS169.2x+1T2016/discussion/forum/i4x-BerkeleyX-CS169_2xb-course-2013_Fall/threads/56a389635fd70904fc000c88