magento-hackathon/Implement-Module-for-prediction.io

Issues

• Prediction.io schema has changed since February, 2014

  #16 opened 10 years ago by edudemy
  0

• Remove when refactored to single id [app/code/community/Magehack/Predictions/Block/Catalog/Product/List/Related.php]

  #14 opened 12 years ago by amacgregor
  0

• Refactor this asap, terrible code hacked together to make it work [app/code/community/Magehack/Predictions/Model/Cron.php]

  #15 opened 12 years ago by amacgregor
  0

• Add a collection limit thorugh configuration [app/code/community/Magehack/Predictions/Block/Catalog/Product/List/Related.php]

  #13 opened 12 years ago by amacgregor
  0

• Review the logic for loading the product collection for this block [app/code/community/Magehack/Predictions/Block/Catalog/Product/List/Related.php]

  #12 opened 12 years ago by amacgregor
  0

• Test for performance [app/code/community/Magehack/Predictions/Model/Observer.php]

  #3 opened 12 years ago by amacgregor
  1

• This method is never called, should add EE support. [app/code/community/Magehack/Predictions/Helper/Adapters/Predictionio.php]

  #11 opened 12 years ago by amacgregor
  0

• This method is never called, should add EE returns support. [app/code/community/Magehack/Predictions/Helper/Adapters/Interface.php]

  #10 opened 12 years ago by amacgregor
  0

• Move into a separate cron job [app/code/community/Magehack/Predictions/Model/Queue.php]

  #8 opened 12 years ago by amacgregor
  0

• Use an iterator(s) here since nested foreachs doesn't necessarily makes sense [app/code/community/Magehack/Predictions/Model/Queue.php]

  #9 opened 12 years ago by amacgregor
  0

• This code should be moved into a separate cron class [app/code/community/Magehack/Predictions/Model/Queue.php]

  #6 opened 12 years ago by amacgregor
  0

• Refactor this code there is no need for doing the join [app/code/community/Magehack/Predictions/Model/Queue.php]

  #7 opened 12 years ago by amacgregor
  0

• Move into a separate cron class [app/code/community/Magehack/Predictions/Model/Queue.php]

  #5 opened 12 years ago by amacgregor
  0

• Test for correct implementation [app/code/community/Magehack/Predictions/Model/Observer.php]

  #4 opened 12 years ago by amacgregor
  0

• Refactor this code to be in a single function [app/code/community/Magehack/Predictions/Model/Observer.php]

  #2 opened 12 years ago by amacgregor
  0