philou/planning-poker

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

# TODO use rails-assets.org instead of jquery-rails to get jquery
Severity: Minor
Found in Gemfile by fixme

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

    # TODO use stream_to instead
Severity: Minor
Found in app/channels/team_channel.rb by fixme

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

    # TODO move some of this in the Team class using events
Severity: Minor
Found in app/controllers/votes_controller.rb by fixme

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

    # TODO The optimistic lock is not great, they could still have race conditions when getting out of the role of animator, maybe emitting raw sql would be better (arel is an sql ast builder)
Severity: Minor
Found in app/models/team.rb by fixme

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

  <% # TODO get rid of these 2 'if' %>
Severity: Minor
Found in app/views/contributors/show.html.erb by fixme

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

# TODO define env var in some special purpose place (check dotenv gem)
Severity: Minor
Found in config/application.rb by fixme

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

# TODO find a way to track annotation comments in files at the root of the repo
Severity: Minor
Found in config/application.rb by fixme

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

    # TODO try to use a plural :votes as usual
Severity: Minor
Found in config/routes.rb by fixme

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

# TODO add a primary key on teams(name)
Severity: Minor
Found in db/todo.rb by fixme

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

# TODO add a primary key on contributors(name, team)
Severity: Minor
Found in db/todo.rb by fixme

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

  # TODO upgrade postgresql to the same as Heroku
Severity: Minor
Found in docker-compose.yml by fixme

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

  # TODO do some polling instead of sleeping for a full second

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

# TODO remove the 'from his browser' from the step def. This is really implementation specific

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

  # TODO Write unit test for the subscription when the action cable tests commits are available (https://github.com/rails/rails/pull/23211 and https://github.com/rspec/rspec-rails/issues/1606)
Severity: Minor
Found in spec/channels/team_channel_spec.rb by fixme

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

# TODO use FactoryGirl to simplify the data setup.

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

    # TODO move to an hexagonal architecture in order to encapsulate team.animator= this would ensure a unique way to update the animator, and we could remove this almost duplicated test https://medium.com/@vsavkin/hexagonal-architecture-for-rails-developers-8b1fee64a613#.c2giyb3mh

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

# TODO find a way to test that we are subscribing with the correct arguments.

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

    # TODO update jasmine test framework version
Severity: Minor
Found in spec/teaspoon_env.rb by fixme

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

    # TODO could we load jasmine through the rails-asset gem source instead of teaspoon ?
Severity: Minor
Found in spec/teaspoon_env.rb by fixme

TODO found
Open
  • Open
  • Confirmed
  • Invalid
  • Wontfix
Cancel

  # TODO put back the default config.server_port = nil when the bug in puma 3.7.0, which overrides command line port with config file is fixed. Currently Teaspoon cannot pass its default port this way. (https://github.com/jejacks0n/teaspoon/issues/512 & https://github.com/puma/puma/issues/1154)
Severity: Minor
Found in spec/teaspoon_env.rb by fixme
Severity
Category
Status