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

Add internal hotjar NPS stylesheet #222

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

annarhughes
Copy link
Member

What changes did you make?

Added the hotjar NPS styles to the public folder instead of trying to import them from Bloom

Why did you make the changes?

Importing/linking an external stylesheet was not working with next.js - there are complexities here and next.js only works well with bundled packages css. Therefore duplicating the hotjar NPS styles in Bloom, as this is a minor and the styles are unlikely to be changed

Copy link

github-actions bot commented Nov 6, 2023

Hi @annarhughes! 👋
Thank you for submitting a pull request for Chayn! We appreciate your contribution and will review your changes as soon as possible. Please read the CONTRIBUTING.md file to learn more about contributing to Chayn. Happy coding!

@annarhughes annarhughes merged commit ec6fd48 into master Nov 6, 2023
4 checks passed
@annarhughes annarhughes deleted the hotjar-nps-stylesheet-internal branch November 6, 2023 14:54
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.

1 participant