redhat-beyond / JobSeeker

https://github.com/redhat-beyond/JobSeeker
MIT License
2 stars 5 forks source link

🚀 Adding Create Post Form #91

Closed paOmer closed 2 years ago

paOmer commented 2 years ago

Added a post creation form to let users have the ability to publish posts. Used django CreateView class.

I tried to not include the post creation html template in this PR in order to not include UI but I had to set a html page for testing purposes.

Future tasks:

Close #92

Yarboa commented 2 years ago

Added a post creation form to let users have the ability to publish posts. Used django CreateView class.

  • Added a CreateView for a new post.
  • Added a redirection function on the post model to be used after creating new post.
  • Set the post creation page url.
  • Added a post creation html template.
  • Added relevant tests.

I tried to not include the post creation html template in this PR in order to not include UI but I had to set a html page for testing purposes.

Future tasks:

Please add an Issue here

  • Adding the ability to create a job-offer post
  • Adding a link to the create new post form
  • Style the form using crispy-forms

Close #92

paOmer commented 2 years ago

Added a post creation form to let users have the ability to publish posts. Used django CreateView class.

  • Added a CreateView for a new post.
  • Added a redirection function on the post model to be used after creating new post.
  • Set the post creation page url.
  • Added a post creation html template.
  • Added relevant tests.

I tried to not include the post creation html template in this PR in order to not include UI but I had to set a html page for testing purposes. Future tasks:

Please add an Issue here

  • Adding the ability to create a job-offer post
  • Adding a link to the create new post form
  • Style the form using crispy-forms

Close #92

Created the issues and edited this PR description to include them.

paOmer commented 2 years ago

@paOmer Nice work, I just do not see any client tests in the PR. When we test View/Form it is not guaranteed the the hole path is not broken from http request to http response

Thanks, Updated the tests accordingly.