Skip to content
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

Suggestion: split up the concept of a "Mentor" and a "Mentor visit" #117

Open
tomduggan85 opened this issue Apr 2, 2018 · 0 comments
Open

Comments

@tomduggan85
Copy link

This is just a suggestion, and it might be too early in the project's life to worry about this (though, on the other hand, since it relates to how data is stored in the database, it might be too late to make this change later):

I noticed that a "Mentor" model currently has data for both a person (name, bio) and data for that person's visit (entry date, permit request status, flight info, etc.). It might make more sense to separate these into two models: A mentor, and a mentor-visit. Mentors could then have a reference to their mentor-visits, and the data model would then let you have one mentor with multiple visits (without having to overwrite their past visit information).

What do you think?
Hope all is well with all of you, and that you're safe.

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

No branches or pull requests

1 participant