Conversation
spencerldixon
force-pushed
the
add_pame_evaluations
branch
3 times, most recently
from
a1659a8 to
6608130
Compare
spencerldixon
force-pushed
the
add_pame_evaluations
branch
from
6608130 to
02a886d
Compare
| @@ -1,3 +1,5 @@ | |||
| require 'erb' | |||
There was a problem hiding this comment.
I kept running into an error that ERB was being used without explicitly being required, this fixes it. Might be a ruby version thing, can't see the harm in being cautious
| module Web; end | ||
| require "web/helpers" | ||
| require "lib/admin_csv_generator" | ||
| require 'sinatra' |
There was a problem hiding this comment.
Again I'm not sure what is this require for exactly
There was a problem hiding this comment.
Same as above, fixes a warning about things being explicitly required for me
|
Work just needs the pame model attributes changing from This work is on hold for the time being so will leave the PR open to be picked up in the future. Pame evaluations should NOT show in the API atm. |
|
@Levia @spencerldixon can either of you shed some light on this very old PR? |
3 participants
No description provided.