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

feat(nuxt): Add Nuxt menu option and install SDK #711

Merged
merged 2 commits into from
Nov 15, 2024

Conversation

andreiborza
Copy link
Member

@andreiborza andreiborza commented Nov 14, 2024

Tested locally with a Nuxt app. Will add tests and e2e tests #710.

#skip-changelog

Closes: #706

Copy link

github-actions bot commented Nov 14, 2024

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against aca2b17

lforst
lforst previously requested changes Nov 14, 2024
Copy link
Member

@lforst lforst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this state, this PR does more bad than good. Let's only merge once the wizard actually does a functioning SDK setup.

@andreiborza
Copy link
Member Author

@lforst merge target is an "epic" branch.

@lforst lforst dismissed their stale review November 14, 2024 17:06

goes into epic branch

Copy link
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work so far! I ran the wizard and things look good. I think it's okay to merge this into the epic branch.

One nit (feel free to do in this PR or in a separate one): I think this is not on you but can we make the filename in red be coloured in cyan? Just for consistency...
image

@andreiborza
Copy link
Member Author

@Lms24 updated to cyan

@andreiborza andreiborza merged commit 7cf7927 into ab/nuxt-wizard Nov 15, 2024
13 checks passed
@andreiborza andreiborza deleted the ab/nuxt-menu-option branch November 15, 2024 08:20
andreiborza added a commit that referenced this pull request Nov 25, 2024
* feat(nuxt): Add Nuxt menu option and install SDK

* Mark sentry dot env file in cyan color
andreiborza added a commit that referenced this pull request Nov 27, 2024
* feat(nuxt): Add Nuxt menu option and install SDK (#711)

* feat(nuxt): Add Nuxt menu option and install SDK

* Mark sentry dot env file in cyan color

* feat(nuxt): Add sdk module to nuxt config and create client and server config files (#713)

* feat(nuxt): Add sdk module to nuxt config and create client and server config files

* Log snippets when user denies overwriting their configs

* feat(nuxt): Add example page/component creation and final messaging (#717)

* feat(nuxt): Add nuxt 3 and 4 e2e test apps (#718)

* feat(nuxt): Add nuxt-3 e2e tests

* Add nuxt-3 e2e tests

* Add describe level

* Add nuxt-4 e2e test app

* Add nuxt 4 e2e test app

* Remove leaking test helper

* change client source map to hidden

* add scoped styles

* Add fallback in case magicast can't parse/modify the nuxt config

* change min version to 3.7.0

* Update src/nuxt/sdk-setup.ts

Co-authored-by: Lukas Stracke <lukas.stracke@sentry.io>

---------

Co-authored-by: s1gr1d <sigrid.huemer@posteo.at>
Co-authored-by: Lukas Stracke <lukas.stracke@sentry.io>
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