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

enh(faces): Avoid O(n) API calls on faces overview #1881

Merged
merged 4 commits into from
Jul 27, 2023

Conversation

marcelklehr
Copy link
Member

@marcelklehr marcelklehr commented Jun 11, 2023

@marcelklehr marcelklehr force-pushed the enh/use-faces-props-in-overview branch from 84ca174 to b9c1174 Compare June 12, 2023 09:14
@marcelklehr
Copy link
Member Author

/compile

@marcelklehr marcelklehr force-pushed the enh/use-faces-props-in-overview branch 2 times, most recently from 75ef862 to 6a4cef9 Compare June 12, 2023 13:35
@marcelklehr
Copy link
Member Author

@artonge Argh, it doesn't accept my manual build. What do we do now?

@artonge
Copy link
Collaborator

artonge commented Jun 12, 2023

Have you npm cied before building?

@marcelklehr
Copy link
Member Author

Yes

@artonge
Copy link
Collaborator

artonge commented Jun 12, 2023

Then might be a difference in node or npm version

@marcelklehr marcelklehr force-pushed the enh/use-faces-props-in-overview branch from 6a4cef9 to fbbfd6a Compare July 27, 2023 11:54
@marcelklehr
Copy link
Member Author

/compile

Signed-off-by: Marcel Klehr <[email protected]>
@marcelklehr marcelklehr merged commit c3a9c36 into master Jul 27, 2023
23 checks passed
@marcelklehr marcelklehr deleted the enh/use-faces-props-in-overview branch July 27, 2023 12:16
@marcelklehr
Copy link
Member Author

🎉

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.

2 participants