-
Notifications
You must be signed in to change notification settings - Fork 810
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
feat: add global search for events, experiments and archives. #1034
Conversation
@wanyoulc Please sign off DCO first. |
Signed-off-by: wanyoulc <15694528183@163.com>
Signed-off-by: wanyoulc <15694528183@163.com>
e7e247d
to
cf29366
Compare
Codecov Report
@@ Coverage Diff @@
## master #1034 +/- ##
==========================================
- Coverage 55.78% 47.89% -7.90%
==========================================
Files 68 72 +4
Lines 4383 4320 -63
==========================================
- Hits 2445 2069 -376
- Misses 1768 2042 +274
- Partials 170 209 +39
Continue to review full report at Codecov.
|
Signed-off-by: wanyoulc <15694528183@163.com>
0e0728f
to
0a5074a
Compare
…or better readability Signed-off-by: wanyoulc <15694528183@163.com>
const [loading, setLoading] = useState(false) | ||
const [e, setEvent] = useState<Event | undefined>(undefined) | ||
|
||
const fetchEvent = () => { |
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.
@fewdan Seems that we need to create an interface to get individual events.
@wanyoulc And also, the new searching styles don't fit the dark mode. PTAL. |
…nd better maintainability add keyboard evnet for triggering search fix some bugs and details Signed-off-by: wanyoulc <15694528183@163.com>
…aos-mesh into update-global-search Signed-off-by: wanyoulc <15694528183@163.com>
998f35d
to
7ac14fa
Compare
Signed-off-by: wanyoulc <15694528183@163.com>
…aos-mesh into update-global-search
Signed-off-by: wanyoulc <15694528183@163.com>
Signed-off-by: wanyoulc <15694528183@163.com>
Signed-off-by: Yue Yang <g1enyy0ung@gmail.com>
Signed-off-by: wanyoulc <15694528183@163.com>
…aos-mesh into update-global-search
add try catch in out of searching process fix some details Signed-off-by: wanyoulc <15694528183@163.com>
c7ef76c
to
9accb3c
Compare
Signed-off-by: wanyoulc <15694528183@163.com>
9accb3c
to
752b40d
Compare
Signed-off-by: Yue Yang <g1enyy0ung@gmail.com>
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
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
/merge |
Your auto merge job has been accepted, waiting for:
|
Please resolve the conflict files. @wanyoulc |
/run-all-tests |
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
…mesh#1034) * feat: add global search for events, experiments and archives. Signed-off-by: wanyoulc <15694528183@163.com> Signed-off-by: wanyoulc <15694528183@163.com>
What problem does this PR solve?
This PR adds a search trigger button, and a global search modal for searching events, experiments, and archives more conveniently.
What is changed and how does it work?
Checklist
Tests
Side effects
Related changes
Does this PR introduce a user-facing change?