-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Workplace Search] Use the correct naming: organizational source, personal dashboard, private source #112426
Conversation
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.
LGTM except for one suggested change. Thanks for doing this! Really sets us up for success in the long term 🎉
..._search/public/applications/workplace_search/views/groups/components/group_manager_modal.tsx
Outdated
Show resolved
Hide resolved
...public/applications/workplace_search/views/groups/components/group_source_prioritization.tsx
Show resolved
Hide resolved
…_search/views/groups/components/group_manager_modal.tsx Co-authored-by: Scotty Bollinger <[email protected]>
💚 Build SucceededMetrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
…sonal dashboard, private source (elastic#112426) * Rename shared sources -> organizational sources on Overview page * Rename shared sources -> organizational sources in groups * Rename shared sources -> organizational sources on Sources pages * Rename shared sources -> organizational sources in filenames and imports * Rename private dashboard to personal dashboard * Rename PERSONAL_SOURCES_PATH -> PRIVATE_SOURCES_PATH * Rename personalSourcesCount -> privateSourcesCount * Rename canCreatePersonalSources -> canCreatePrivateSources * Rename the remaining personal_source -> private_source * Autofix i18n * Update x-pack/plugins/enterprise_search/public/applications/workplace_search/views/groups/components/group_manager_modal.tsx Co-authored-by: Scotty Bollinger <[email protected]> Co-authored-by: Scotty Bollinger <[email protected]>
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
…sonal dashboard, private source (#112426) (#112482) * Rename shared sources -> organizational sources on Overview page * Rename shared sources -> organizational sources in groups * Rename shared sources -> organizational sources on Sources pages * Rename shared sources -> organizational sources in filenames and imports * Rename private dashboard to personal dashboard * Rename PERSONAL_SOURCES_PATH -> PRIVATE_SOURCES_PATH * Rename personalSourcesCount -> privateSourcesCount * Rename canCreatePersonalSources -> canCreatePrivateSources * Rename the remaining personal_source -> private_source * Autofix i18n * Update x-pack/plugins/enterprise_search/public/applications/workplace_search/views/groups/components/group_manager_modal.tsx Co-authored-by: Scotty Bollinger <[email protected]> Co-authored-by: Scotty Bollinger <[email protected]> Co-authored-by: Vadim Yakhin <[email protected]> Co-authored-by: Scotty Bollinger <[email protected]>
Summary
This PR renames several things in the Workplace Search codebase:
Enterprise Search needs to be run with these changes included https://github.com/elastic/ent-search/pull/4670 for this PR to work.
Many changes only touched variable names, but here are some screenshots with the UI changes:
Checklist