Skip to content
This repository has been archived by the owner on Jun 4, 2020. It is now read-only.

Added Association 42 (.fr) #37

Closed
wants to merge 2 commits into from
Closed

Conversation

sruon
Copy link

@sruon sruon commented Nov 19, 2013

Association 42 is a new IT school located in Paris, France offering a free 3 year tuition to its students.
42 is entirely project-driven and puts the focus on teamwork.

Students are offered login@student.42.fr aliases.
Staff uses login@staff.42.fr addresses.

www.42.fr

@johndbritton
Copy link
Collaborator

I'd prefer if we limited the inclusion to the students.42.fr subdomain. Also, you should include a newline at the end of your file.

@sruon
Copy link
Author

sruon commented Nov 21, 2013

Hello,

I have limited the inclusion to student.42.fr subdomain and added the new line.

Thanks a lot.

@johndbritton
Copy link
Collaborator

@leereilly Is this the right format? I think the file needs to be 42.fr/student, but I'm not 100% sure.

@afeld
Copy link
Collaborator

afeld commented Dec 3, 2013

@leereilly bump!

@leereilly
Copy link
Owner

Thanks for the contribution @sruon! I've taken a while to think it over, but I'm still torn on this one. While I think students at 42.fr should definitely receive things like academic discounts (and free :octocat: bronze accounts!), I'm unable to merge this pull request right now.

The original intent for this gem was to hold official country/state-recognized accredited schools only. I'd be open to distinguishing schools somehow in future - perhaps adding a classification of e.g. university, college, non-accredited, etc.

42.fr looks perfectly legit, but I'd be worried that adding this'd set precedent for adding other non-accredited schools... and open up the doors to abuse. What do you think @johndbritton & @afeld?

@johndbritton
Copy link
Collaborator

Swot is one of a few ways that we validate student status, so it won't hurt us if this doesn't get merged. We can still offer the discounts direct to students at 42.

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.

None yet

4 participants