-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
v4.0 | Update docs #32
Conversation
@rensio Nu met Chakra-UI example 🔥 |
@@ -0,0 +1,16 @@ | |||
{ | |||
"name": "@freshheads/cookie-guard-example-chakra-ui", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is het idee hier dat je deze package.json niet in deze library build maar puur als voorbeeld gebruikt ?
Of kun je dit met een script in de main package.json laten builden en daar een server voor laten draaien?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ja inderdaad puur als voorbeeld, door dit zo op te zetten met eigen node_modules is het makkelijker om de package te developen. Je krijgt gewoon je imports en type checking etc. En om de example te gebruiken kun kijken wat er in de package.json staat. (Packages die buiten deze example niet nodig zijn, zoals chakra-ui)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moeten alleen straks nog even checken of dat goed gaat, dat bijvoorbeeld niet chakra-ui als dependency van de cookie-guard komt.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok. Het lijkt mij inderdaad wel wenselijk dat we niet teveel dependencies moeten bijhouden puur voor de voorbeeld code. Maar wel handig als je ergens wat hebt staan natuurlijk als je toch in een Chakra werkt.
Het voorbeeld wat je ziet wanneer je de dev server draait gaat nu nog steeds via de css bestanden toch en niet met de chakra dependencies?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ja dat klopt, deze example kan je niet standaard builden vanuit de package
No description provided.