Api controller

Опять сиплюсизм

return true if true == true

Надо так:
def is_project_achievement?
  !!self.project
end

Это если тебя волнует вообще чтобы результат был прям булевский. Если не волнует (а в данном случае не волнует) - то два !! не нужны. А учитывая упрощение весь метод не нужен, можно прямо по месту писать if project