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

Store periods in each section #50

Open
kburk1997 opened this issue Apr 12, 2017 · 6 comments
Open

Store periods in each section #50

kburk1997 opened this issue Apr 12, 2017 · 6 comments

Comments

@kburk1997
Copy link
Member

No description provided.

@kburk1997
Copy link
Member Author

Right now you cannot add periods to sections.

@Bad-Science
Copy link
Member

Can periods be a part of the section form since they arent their own resource? I think this would be more convenient than having a separate form too.

@kburk1997
Copy link
Member Author

That's what I was thinking, I have no idea how to go about it though

@Bad-Science
Copy link
Member

Bad-Science commented Apr 19, 2017

@kburk1997 treat periods just as another attribute of sections that is an array of hashes. e.g.

section:  { 
    id: 1, name: '02', ..., periods: [
      { startTime: '800',  endtime: '850', day: 2 }, 
      { startTime: '800',  endtime: '850', day: 5 }
    ]
  }

I'm not sure if the form helpers can do this. You may need to write your own form.

@kburk1997
Copy link
Member Author

Right now, there are two parts to this issue:

  • Create the design for the form (frontend)

  • Create the actual variable for storing period (backend)

I suggest splitting this issue into two issues for clarity.

@kburk1997 kburk1997 changed the title Create form for period Store periods in each section Oct 3, 2017
@kburk1997 kburk1997 removed the frontend label Oct 3, 2017
@kburk1997
Copy link
Member Author

kburk1997 commented Oct 3, 2017

See issue #65 for the actual frontend issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants