Feature 217 view issue threads without logging in

[ashfaq1934]

Threads app update

• Created a new view to display the feed
• Created views for adding and removing Civis
• Created tests for new views
• Updated feed template and view to display threads without having to register an account and removed placeholder from thread.html
• Removed automatic site wide redirect in settings.py to allow the login view to redirect to different URLs. Modified login.html form to use next value if provided, and default to the base URL if not.
• Moved landing page template to another view
• Bug-fixes for less files not being found in about.html and how_it_works.html
• Modified new thread button to direct the user to login if they haven't been authenticated, once they've logged in, they'll be redirected back to the thread
• Fixed errors produced by ThreadDetailViewset and removed JSON encoding
• Added section for messages in global_nav.html
• Modified JavaScript in thread.html for card display and form submission. commented out code that was producing JavaScript errors and applied formatting fixes with flake8
• Modified tests for landing and feed templates
• closes View Issue Threads without Registering Account #217
• relates to Port threads app templates to Django template syntax #1152

[ashfaq1934]

The Linting in CI Workflow fails because of the Black formatting changes that might need to be applied


Should I apply Black formatting to these files so that the workflow passes?

[brylie]

Yes, all files should be formatted with ‘black’

[codeclimate]

Code Climate has analyzed commit ca4ccfc and detected 0 issues on this pull request.

View more on Code Climate.

[ashfaq1934]

Here's a demo of the changes that have been implemented. Users will be able to create and delete civis. They can view threads but if they attempt to create a civi they will be redirected to the login url, after they login, they'll be redirected back to the thread they viewed.

demo_1.mp4

I've managed to get the workflow to pass by editing accounts/views.py which satisfied the linter.

Let me know if there's anything missing, hope this helps.