Skip to content

SSR - Add more component tests#899

Merged
devongovett merged 2 commits into
mainfrom
ssr
Aug 11, 2020
Merged

SSR - Add more component tests#899
devongovett merged 2 commits into
mainfrom
ssr

Conversation

@dannify
Copy link
Copy Markdown
Member

@dannify dannify commented Aug 6, 2020

Related to #760, #699
Follow up from #835

This is to cover the basic rendering of the rest of our components in a server rendered environment. Missing Icon for now.

Copy link
Copy Markdown
Member

@snowystinger snowystinger left a comment

Choose a reason for hiding this comment

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

any chance we can add it to our workflows for circle?

@dannify
Copy link
Copy Markdown
Member Author

dannify commented Aug 6, 2020

Do you want to ever have our CI pass ever again?

any chance we can add it to our workflows for circle?

@dannify
Copy link
Copy Markdown
Member Author

dannify commented Aug 6, 2020

The plan is to add it to CI when they all pass 😸

Copy link
Copy Markdown
Member

@LFDanLu LFDanLu left a comment

Choose a reason for hiding this comment

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

Looks good to me, what do you think about adding a SSR test template to the plop templates?

@adobe-bot
Copy link
Copy Markdown

Build successful! 🎉

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.

5 participants