-
Notifications
You must be signed in to change notification settings - Fork 384
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 web viewer feature flags #8295
Changes from all commits
02b8516
805465b
be4c4b9
315790d
a6d9637
37cd2dd
d4de72d
6d1b44b
b7e8d58
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -31,7 +31,8 @@ function buildWebViewer(mode) { | |
"cargo run -p re_dev_tools -- build-web-viewer", | ||
modeFlags, | ||
"--target no-modules-base", | ||
"--features grpc", | ||
"--no-default-features", | ||
"--features grpc,map_view", // no `analytics` | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. why not? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this is for the NPM package, we don't want users' applications which embed the web viewer this way to send analytics to us. they will only do that if they embed the web viewer as an iframe |
||
"-o rerun_js/web-viewer", | ||
].join(" "), | ||
); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't easier to keep the default features and just add on grpc?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to be more explicit here, because you're not going to type the long command