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

Continue the work on cross-network / cross-paratime search #449

Merged
merged 8 commits into from Jun 7, 2023

Conversation

csillag
Copy link
Contributor

@csillag csillag commented May 26, 2023

This is the successor of #438; the remaining parts since #447 has been extracted and merged.

@github-actions
Copy link

github-actions bot commented May 26, 2023

Deployed to Cloudflare Pages

Latest commit: bc8f0366400d53cc5fcad25b23971d16a4928907
Status:✅ Deploy successful!
Preview URL: https://187e62d9.oasis-explorer.pages.dev

@csillag
Copy link
Contributor Author

csillag commented May 26, 2023

@donouwens says (on Slack):

Looks good to me!

So... finally we can get on with the technical review, yay!

@csillag csillag requested review from lukaw3d and buberdds May 26, 2023 07:26
@csillag csillag force-pushed the csillag/cross-chain-search-reloaded branch from f90f59a to 4796e7f Compare May 26, 2023 07:26
@lukaw3d lukaw3d force-pushed the csillag/cross-chain-search-reloaded branch 3 times, most recently from d6dd2db to b0ee3e6 Compare May 26, 2023 12:14
src/locales/en/translation.json Outdated Show resolved Hide resolved
src/app/pages/SearchResultsPage/index.tsx Outdated Show resolved Hide resolved
src/app/pages/SearchResultsPage/index.tsx Outdated Show resolved Hide resolved
src/app/pages/SearchResultsPage/ResultsFilteredThemed.tsx Outdated Show resolved Hide resolved
src/app/pages/SearchResultsPage/ResultsFilteredThemed.tsx Outdated Show resolved Hide resolved
src/app/pages/SearchResultsPage/ResultsFilteredThemed.tsx Outdated Show resolved Hide resolved
src/app/pages/SearchResultsPage/ResultsElsewhere.tsx Outdated Show resolved Hide resolved
@csillag csillag force-pushed the csillag/cross-chain-search-reloaded branch 3 times, most recently from 32a5af7 to 0dc37ae Compare May 27, 2023 01:36
@csillag csillag requested a review from lukaw3d May 27, 2023 02:21
@csillag csillag marked this pull request as draft May 27, 2023 06:45
@csillag csillag force-pushed the csillag/cross-chain-search-reloaded branch from 5ebcba6 to e1b4e8c Compare May 27, 2023 07:08
@csillag csillag marked this pull request as ready for review May 27, 2023 07:08
@csillag csillag force-pushed the csillag/cross-chain-search-reloaded branch from e1b4e8c to 503f712 Compare May 27, 2023 07:10
@csillag
Copy link
Contributor Author

csillag commented May 27, 2023

I separated the components for global search results and scoped search results. I think it's easier to read now.

@csillag csillag force-pushed the csillag/cross-chain-search-reloaded branch 8 times, most recently from 696ed38 to f8ffed8 Compare May 28, 2023 18:20
@csillag csillag force-pushed the csillag/cross-chain-search-reloaded branch from f8ffed8 to 8d7ae68 Compare May 29, 2023 19:37
@csillag csillag changed the title Continue the work on cross-site / cross-paratime search Continue the work on cross-network / cross-paratime search May 30, 2023
@csillag csillag force-pushed the csillag/cross-chain-search-reloaded branch 3 times, most recently from 7f97e16 to a094f63 Compare June 6, 2023 12:01
@csillag csillag force-pushed the csillag/cross-chain-search-reloaded branch 6 times, most recently from ebc16b7 to 635302c Compare June 7, 2023 16:05
@csillag csillag force-pushed the csillag/cross-chain-search-reloaded branch from c0252d9 to f060f9c Compare June 7, 2023 22:53
Filtering has been moved up one layer, so it makes no sense
to call this file Filtered anymore.
@csillag
Copy link
Contributor Author

csillag commented Jun 7, 2023

I have now cut this to multiple commits, and will merge as soon as the tests are finished.

@csillag csillag merged commit 1d57de2 into master Jun 7, 2023
6 checks passed
@csillag csillag deleted the csillag/cross-chain-search-reloaded branch June 7, 2023 22:59
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.

None yet

2 participants