Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Rename "global search" to "unified search"#42094

Merged
susnux merged 2 commits intomasterfrom
refactor-global-search-to-unified-search
Dec 12, 2023
Merged

Rename "global search" to "unified search"#42094
susnux merged 2 commits intomasterfrom
refactor-global-search-to-unified-search

Conversation

@nfebe
Copy link
Contributor

@nfebe nfebe commented Dec 7, 2023

  • Changes appearances of "Global search" to "Unified search" in UI
  • Refactors code, to remove usage of term "GlobalSearch" in files and code structure
  • Rename old unified search to legacy-unified-search

@nfebe nfebe added 3. to review Waiting for reviews feature: search labels Dec 7, 2023
@AndyScherzinger AndyScherzinger added this to the Nextcloud 29 milestone Dec 7, 2023
@AndyScherzinger
Copy link
Member

AndyScherzinger commented Dec 7, 2023

/backport to stable28

@nfebe nfebe force-pushed the refactor-global-search-to-unified-search branch 2 times, most recently from 6a212f3 to b2464eb Compare December 7, 2023 12:29
@sorbaugh sorbaugh self-requested a review December 7, 2023 12:49
@nfebe nfebe force-pushed the refactor-global-search-to-unified-search branch from b2464eb to 84d1dbd Compare December 7, 2023 13:05
@nfebe
Copy link
Contributor Author

nfebe commented Dec 7, 2023

/compile

@nfebe nfebe mentioned this pull request Dec 7, 2023
Merged
@nfebe nfebe force-pushed the refactor-global-search-to-unified-search branch 6 times, most recently from c9119b5 to 259f671 Compare December 11, 2023 13:36
susnux
susnux reviewed Dec 11, 2023
View reviewed changes
@nfebe nfebe force-pushed the refactor-global-search-to-unified-search branch from 259f671 to 11467f2 Compare December 11, 2023 15:13
@nfebe nfebe requested a review from susnux December 11, 2023 15:16
susnux
susnux requested changes Dec 11, 2023
View reviewed changes
Copy link
Contributor

@susnux susnux left a comment

Choose a reason for hiding this comment

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

Tested and no search is loaded at all because of this error:

Uncaught Error: Could not find initial state limit-default of unified-search
    r index.es.mjs:13
    54201 UnifiedSearchService.js:30
    Webpack 5
        o
        c
        O
        <anonymous>
        <anonymous>
index.es.mjs:13:14

@nfebe
Copy link
Contributor Author

nfebe commented Dec 11, 2023

Tested and no search is loaded at all because of this error:

Uncaught Error: Could not find initial state limit-default of unified-search
r index.es.mjs:13
54201 UnifiedSearchService.js:30
Webpack 5
o
c
O


index.es.mjs:13:14

This change does not contain the build, did you watch or build? I can't see this?

@nfebe nfebe force-pushed the refactor-global-search-to-unified-search branch from 11467f2 to 1de5ad6 Compare December 11, 2023 15:33
emoral435
emoral435 approved these changes Dec 11, 2023
View reviewed changes
Copy link
Contributor

@emoral435 emoral435 left a comment

Choose a reason for hiding this comment

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

Tested this out in my dev setup and works, also looked over the code and it looks good to me 👍

@emoral435
Copy link
Contributor

emoral435 commented Dec 11, 2023

Tested and no search is loaded at all because of this error:

I think it is because of not running npm build or watch, as @fenn-cs mentioned. After running watch, the app works as expected on my dev setup with no error

@nfebe
Copy link
Contributor Author

nfebe commented Dec 12, 2023

@susnux Since, you left a change request on this, I would like you to take a look one more time, thank you!

@nfebe nfebe requested a review from susnux December 12, 2023 18:49
@nfebe @susnux
Rename "global search" to "unified search"
0b171f6 
- Changes appearances of "Global search" to "Unified search" in UI
- Refactors code, to remove usage of term "GlobalSearch" in files and code
 structure
- Rename old unified search to `legacy-unified-search`

Signed-off-by: fenn-cs <[email protected]>
@susnux susnux force-pushed the refactor-global-search-to-unified-search branch from 1de5ad6 to 0b171f6 Compare December 12, 2023 19:35
@susnux
chore: Compile assets
2f66897 
Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux merged commit c734a18 into master Dec 12, 2023
@susnux susnux deleted the refactor-global-search-to-unified-search branch December 12, 2023 20:13
@nfebe
Copy link
Contributor Author

nfebe commented Dec 12, 2023

/backport to stable28

@nfebe nfebe mentioned this pull request Dec 12, 2023
Merged
@nfebe
Copy link
Contributor Author

nfebe commented Dec 13, 2023

/backport to stable28

@Pytal Pytal mentioned this pull request Dec 14, 2023
Merged
2 tasks
@nfebe nfebe mentioned this pull request Dec 21, 2023
Merged
@blizzz blizzz mentioned this pull request Mar 5, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AndyScherzinger AndyScherzinger AndyScherzinger approved these changes

@susnux susnux susnux approved these changes

@artonge artonge Awaiting requested review from artonge

@Pytal Pytal Awaiting requested review from Pytal

@sorbaugh sorbaugh Awaiting requested review from sorbaugh

+1 more reviewer

@emoral435 emoral435 emoral435 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

3. to review Waiting for reviews feature: search

Projects

📁 Files team
Archived in project

Milestone

Nextcloud 29

Development

Successfully merging this pull request may close these issues.

5 participants

@nfebe @AndyScherzinger @emoral435 @susnux @skjnldsv