search

cjohansen / portfolio

Eclipse Public License 1.0
236 stars 14 forks source link

Helix support #2

Closed rome-user closed 1 year ago

rome-user commented 1 year ago

As promised, here is the PR that adds support for Helix.

cjohansen commented 1 year ago

Great stuff! Seeing as there is nothing helix-specific in here, the namespace might as well be portfolio.react. Keeping the helix example is fine.

Could you also add a note in the relevant section of the Readme, so people know it's available? 😊

rome-user commented 1 year ago

OK I have renamed portfolio.helix to portfolio.react and updated the sample to reflect this.