Ignore tmp directory
Break apart connection preprocessors and control flow from the view not the template
Create a namespace for views
Separate connection view helpers into their own files
assign the session rather than accessing it from the request every time
Move disconnection logic into helpers
Move lastfm connection logic into helper
Move lastfm connection url generation into the main views
Make the lastfm_connection helper a class to contain session and api
Move the lastfm connection helper into the middleware
Handle failed lastfm authentication
Move url knowledge out of the lastfm connection helper
Rename LastfmApp middleware to LastfmApi and rename the api instance to lastfm_api
Separate lastfm connection ui logic from main index method
Better separate the Spotify connection logic
- Move the form out of the spotify_connection module and into the main view
- Turn the spotify_connection module into the SpotifyConnectionHelper class which mostly matches the interface of the LastfmConnectionHelper
- Move Spotify connection logic out of the form validation and into the SpotifyConnectionHelper
- Only set up the Spotify session once in manage.py
- Add the middleware to the settings so it doesn't have to be set on a per-view basis
Remove the necessity for showSpotifyForm
Seperate form processing from the spotify_connection_ui_context method
Move knowledge of the spSession out of the view and into the class that creates it
Refactor common connection helper logic into a subclass
Use a non-garbage session key for the lastfm session storage