search

scottyzen / woonuxt

Static e-commerce powered by WooCommerce & Nuxt
https://woonuxt.com
GNU General Public License v3.0
574 stars 139 forks source link

Minor issues - Introduce a new Star config #173

Closed alexookah closed 3 weeks ago

alexookah commented 1 month ago

Hello I found some minor issues

1: https://github.com/scottyzen/woonuxt/blob/master/woonuxt_base/app/pages/index.vue#L3`

name is not defined in app.config.ts
I think you meant to use siteName.
Also maybe in ogDescription you meant to use the shortDescription?

2: Would you conside another config option? Show star rating? (in product and in products grid view?)

If you agree I can make a PR for these changes.

alexookah commented 4 weeks ago

Created PR requests for this https://github.com/scottyzen/woonuxt/pull/176 https://github.com/scottyzen/woonuxt/pull/175

alexookah commented 3 weeks ago

This issue can close now. PR's have been merged