-
Notifications
You must be signed in to change notification settings - Fork 19
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(website-frontend): add new repositories page #519
Conversation
b8c03a4
to
a749a57
Compare
e7a7c12
to
676a51b
Compare
5fb9e55
to
5662007
Compare
It seems that I cannot run the e2e tests anymore, can you run them @hugo-vrijswijk? (I believe it might be because we are using npm workspaces now). |
Locally? I can still run them (though I'm getting a lot of errors, maybe there should be something in the contributing.md on how to run them 😄). In CI they're running fine too. What is not working for you? |
I get the following (tested with 2 different machines, and with a fresh clone):
It seems like the command used to run tests (in the |
Got the tests running and made a new issue for improving this cumbersome workflow: #550, still need to fix the actual tests + add the badge config thingy back on this page. |
packages/website-frontend/src/app/repository/repository-page/repository-page.component.ts
Outdated
Show resolved
Hide resolved
6cacedd
to
715aca6
Compare
68e691f
to
1383b97
Compare
1383b97
to
1fb75d6
Compare
Done now |
Minimal UX improvements, but it will fit better with the current styling.