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

fix(component): add docs overview #2444

Merged

Conversation

BioPhoton
Copy link
Contributor

@BioPhoton BioPhoton commented Mar 20, 2020

Added link to docs intro page. Created content for the components index pag and linked let and push docs.

related to #2441 closes #2442

@ngrxbot
Copy link
Collaborator

ngrxbot commented Mar 20, 2020

Preview docs changes for af0e3ec at https://previews.ngrx.io/pr2444-af0e3ec/

BioPhoton and others added 2 commits March 29, 2020 14:46
Co-Authored-By: Tim Deschryver <28659384+timdeschryver@users.noreply.github.com>
@BioPhoton BioPhoton force-pushed the 2442_add-docs-overview-page-content branch from b073afe to af0e3ec Compare March 29, 2020 12:48
@BioPhoton
Copy link
Contributor Author

Hi @brandonroberts

After syncing with master I run into some cypress related problems:
The cypress npm package is installed, but the Cypress binary is missing.

Not sure how this is related to the PR but if I can do anything please let me know.

Copy link
Member

@timdeschryver timdeschryver left a comment

Choose a reason for hiding this comment

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

LGTM
Fyi, about the build I was noticing a lot of new builds were failing, I was trying it out here
Somehow that fixed it for all branches (after a retrigger)? Perhaps with the default cache key that now has a cache?

@brandonroberts brandonroberts merged commit a279dd1 into ngrx:master Mar 29, 2020
@BioPhoton BioPhoton deleted the 2442_add-docs-overview-page-content branch March 30, 2020 20:40
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.

@ngrx/component Add docs overview page content
4 participants