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

[DO NOT MERGE] - Review app infinite disco #14881

Closed
wants to merge 16 commits into from

Conversation

egdbear
Copy link
Contributor

@egdbear egdbear commented Nov 21, 2024

The type of this PR is: Feat

This PR solves [PROJECT-XX]

Description

https://infinite-disco.artsy.net/infinite-discovery

This PR is the POC of the Elastic Search recommendations.

Screenshot 2024-11-21 at 11 32 38 AM
Screenshot 2024-11-21 at 11 32 48 AM

@ArtsyOpenSource
Copy link

ArtsyOpenSource commented Dec 11, 2024

Warnings
⚠️

Routes added to routes.tsx should have a corresponding cypress.js smoke test. See the cypress/e2e folder for examples.

Generated by 🚫 dangerJS against d7c5860

Copy link

relativeci bot commented Dec 11, 2024

#1288 Bundle Size — 8.96MiB (+0.07%).

d7c5860(current) vs 5ea69d4 main#1283(baseline)

Warning

Bundle contains 14 duplicate packages – View duplicate packages

Bundle metrics  Change 5 changes Regression 1 regression
                 Current
#1288
     Baseline
#1283
Regression  Initial JS 3.65MiB(+0.02%) 3.65MiB
No change  Initial CSS 0B 0B
Change  Cache Invalidation 45.58% 39.32%
Change  Chunks 104(+0.97%) 103
Change  Assets 107(+0.94%) 106
Change  Modules 5836(+0.03%) 5834
No change  Duplicate Modules 530 530
No change  Duplicate Code 4.03% 4.03%
No change  Packages 266 266
No change  Duplicate Packages 13 13
Bundle size by type  Change 2 changes Regression 2 regressions
                 Current
#1288
     Baseline
#1283
Regression  JS 8.81MiB (+0.05%) 8.81MiB
Regression  Other 144.68KiB (+0.92%) 143.36KiB

Bundle analysis reportBranch review-app-infinite-discoProject dashboard


Generated by RelativeCIDocumentationReport issue

package.json Outdated
@@ -147,6 +147,7 @@
"lodash": "^4.17.21",
"luxon": "^3.2.1",
"map-cursor-to-max": "^1.0.0",
"mathjs": "^14.0.0",
Copy link
Contributor Author

@egdbear egdbear Dec 11, 2024

Choose a reason for hiding this comment

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

this will be removed once we have query BE for open search on the metaphysics cc @nickskalkin

@egdbear egdbear force-pushed the review-app-infinite-disco branch from 36a4a39 to f0fc4ec Compare December 16, 2024 13:09
@egdbear
Copy link
Contributor Author

egdbear commented Dec 16, 2024

Closing in favor of #14997

@egdbear egdbear closed this Dec 16, 2024
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.

3 participants