Skip to content
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

Merged
merged 2 commits into from
Oct 13, 2023
Merged

Conversation

sunnyfeijen
Copy link
Collaborator

No description provided.

@sunnyfeijen
Copy link
Collaborator Author

@rensio Nu met Chakra-UI example 🔥

@@ -0,0 +1,16 @@
{
"name": "@freshheads/cookie-guard-example-chakra-ui",
Copy link
Member

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?

Copy link
Collaborator Author

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)

Copy link
Collaborator Author

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.

Copy link
Member

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?

Copy link
Collaborator Author

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

@sunnyfeijen sunnyfeijen merged commit a283af1 into freshheads:develop Oct 13, 2023
@sunnyfeijen sunnyfeijen deleted the v4.0.0-alpha.4 branch October 13, 2023 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants