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

#P09 As a participant - I would like to submit my entry. #87

Merged

Conversation

ClaireLally8
Copy link
Contributor

@ClaireLally8 ClaireLally8 commented Oct 20, 2020

Description

Creates the app for the submissions of completed hackathon project. Utilises the HackProject model and is filled out with a form by the users.

  • Created a forms.py file within the submissions app which then is used by the crispy-forms template to fill the model.
  • Two templates - submit.html and success.html
  • A view that checks if the request is a GET or POST. If it's a GET request, it will render the submit.html page, if its a post, it will take the data from the form, checks if it's valid, saves it into the model and then renders the success.html page.

Pull request type

Testing

Manual testing by checking it functions.
Automated testing - * crickets *

Additional Information

Will need the templates edited as these are not finalised. Found within the submissions/templates/submissions folder :)

Does this introduce a breaking change

  • Yes
  • No

@TravelTimN TravelTimN linked an issue Oct 21, 2020 that may be closed by this pull request
Copy link
Collaborator

@TravelTimN TravelTimN left a comment

Choose a reason for hiding this comment

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

Looking great!
A few minor adjustments outlined below! 🦗

main/settings.py Outdated Show resolved Hide resolved
main/settings.py Outdated Show resolved Hide resolved
submissions/forms.py Outdated Show resolved Hide resolved
submissions/forms.py Outdated Show resolved Hide resolved
submissions/forms.py Outdated Show resolved Hide resolved
submissions/forms.py Outdated Show resolved Hide resolved
submissions/urls.py Outdated Show resolved Hide resolved
submissions/views.py Outdated Show resolved Hide resolved
submissions/views.py Show resolved Hide resolved
submissions/views.py Outdated Show resolved Hide resolved
hackathon/models.py Outdated Show resolved Hide resolved
hackathon/models.py Outdated Show resolved Hide resolved
submissions/apps.py Outdated Show resolved Hide resolved
submissions/views.py Outdated Show resolved Hide resolved
@stefdworschak
Copy link
Member

@ClaireLally8 sorry, I did not want to pile on and be the 3rd person to review your pull request, but since we were talking about your PR a bit I would be amiss not to review it for you!! Also, only for a few minor things I think it looks really good! 🥇

@ClaireLally8
Copy link
Contributor Author

Thanks y'all @stefdworschak @TravelTimN @Eventyret I guess I'll fix these at some stage today ;) (but please, be perfectionists, I need to get better at the refining stuff!)

@ClaireLally8
Copy link
Contributor Author

If you don't approve this *shakes fist *

Copy link
Member

@stefdworschak stefdworschak left a comment

Choose a reason for hiding this comment

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

@ClaireLally8 gonna approve this now since you are threatening me (yes, I am referring to the fist shaking lol). Please have a look at my final few comments and I will merge it when you revert back.

hackathon/models.py Outdated Show resolved Hide resolved
hackathon/models.py Show resolved Hide resolved
hackathon/models.py Outdated Show resolved Hide resolved
submission_time = models.DateTimeField(auto_now_add=True)
speaker_name = models.CharField(default="", max_length=225)
share_permission = models.BooleanField(default=True)
Copy link
Member

Choose a reason for hiding this comment

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

Do we need this here? Or more specifically can the person submitting the project legally agree to the sharing permissions for all team members? I would say we put something in the T&Cs when signing up that they need to confirm that any work that is produced as part of participating on this platform can be shared. @TravelTimN what do you think about this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hackathon/models.py Outdated Show resolved Hide resolved
@stefdworschak
Copy link
Member

@ClaireLally8 @TravelTimN I am going to merge this now and will create a follow up issue for ourselves to decide this.

@stefdworschak stefdworschak merged commit 7aca166 into Code-Institute-Community:master Oct 24, 2020
@stefdworschak stefdworschak added the hacktoberfest-accepted Accepted PR during Hacktoberfest label Oct 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Accepted PR during Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

#P09. As Participant, I would like to submit our Hackathon entry.
4 participants