Instead of converting `challenge` into an instance variable (@challenge), how about simply passing in @challenge into `upload?` method. Within upload, you can get the id by calling `challenge.id`. 

Otherwise, looks good!