search

prmr / Creco

Recommendation System for Consumer Products
Apache License 2.0
6 stars 2 forks source link

Use Template #102

Closed ceipher closed 10 years ago

ceipher commented 10 years ago

Use template to define the content of product section, to be used for both attribute refreshing and product initial loading.

ceipher commented 10 years ago

@MariamN do you think we need the new attribute chart (like the chart for overall score) to be displayed when you enter the product page? Default seems to use only "overall score "

ceipher commented 10 years ago

@enewe101 now it looks that using same template probably does not make sense any more since the view after attributes change is different from initial product view. If they need to be the same, then probably we should move everything into javascript, as Mariam added recently. What do you think

MariamN commented 10 years ago

I think since the products initially are sorted by the overall score it make since to show that to the user.

On Apr 18, 2014, at 7:24 PM, ceipher notifications@github.com wrote:

@MariamN do you think we need the new attribute chart (like the chart for overall score) to be displayed when you enter the product page? Default seems to use only "overall score "

— Reply to this email directly or view it on GitHub.

ceipher commented 10 years ago

ok. so now both views should be consistent..

asutcl commented 10 years ago

Personally, I think the views should be the same. The first screen, once you selected the category, could make the same call as the user putting the overall attribute to the max and all others to 0. That would make the views consistent and would have the desired behaviour. Does that sound reasonable?

enewe101 commented 10 years ago

yup that makes the most sense to me On Apr 18, 2014 8:35 PM, "asutcl" notifications@github.com wrote:

Personally, I think the views should be the same. The first screen, once you selected the category, could make the same call as the user putting the overall attribute to the max and all others to 0. That would make the views consistent and would have the desired behaviour. Does that sound reasonable?

— Reply to this email directly or view it on GitHubhttps://github.com/prmr/Creco/issues/102#issuecomment-40855786 .

ceipher commented 10 years ago

Agree. I will implement in this way.

ceipher commented 10 years ago

Done