Skip to content

Commit

Permalink
remove all TODOs from 01_fyyur/forms.py since they are really confusi…
Browse files Browse the repository at this point in the history
…ng and not checked for anyway"
  • Loading branch information
Matthew Ludwig committed Dec 16, 2019
1 parent a9ba439 commit faa0c81
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions projects/01_fyyur/starter_code/forms.py
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,6 @@ class VenueForm(Form):
'image_link'
)
genres = SelectMultipleField(
# TODO implement enum restriction
'genres', validators=[DataRequired()],
choices=[
('Alternative', 'Alternative'),
Expand Down Expand Up @@ -181,14 +180,12 @@ class ArtistForm(Form):
]
)
phone = StringField(
# TODO implement validation logic for state
'phone'
)
image_link = StringField(
'image_link'
)
genres = SelectMultipleField(
# TODO implement enum restriction
'genres', validators=[DataRequired()],
choices=[
('Alternative', 'Alternative'),
Expand All @@ -213,8 +210,5 @@ class ArtistForm(Form):
]
)
facebook_link = StringField(
# TODO implement enum restriction
'facebook_link', validators=[URL()]
)

# TODO IMPLEMENT NEW ARTIST FORM AND NEW SHOW FORM

1 comment on commit faa0c81

@Thalrion
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a student who is also using a TODO extension for VS Code, these TODO´s were really missleading & annoying. They are not part of the rubic nor the README nor the course work.

Please sign in to comment.