Closed j24m closed 6 months ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
status-app | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Nov 10, 2023 10:13am |
- The query should persist in the search bar after sharing.
- The API should be
search?q=dark%20mode
orsearch?q=dark+mode
(That's what Google use)
/issues?q=<some-key>%3A<dynamic-string>
q=search%3Adark+mode
Please add tests for your feature.
- The query should persist in the search bar after sharing.
- The API should be
search?q=dark%20mode
orsearch?q=dark+mode
(That's what Google use)
- Thankyou for pointing this out, I will persist the search query in search bar.
- Coming to your second point I have constructed the query param as per discussion with @ankushdharkar, here is the screenshot for the same : -So for issues it should be
/issues?q=<some-key>%3A<dynamic-string>
- Example :
q=search%3Adark+mode
For this case, it looks good.
You haven't added a test case to verify whether the changes are actually reflected in the search URL or not when the input field is changed.
Done.
The search should show closest suggestion of query EX - If user search
Dark mod
missinge
, the result should show fordark mode
currently it is showing the exact value to query
Hello, regarding this comment of yours Gaurav : https://github.com/Real-Dev-Squad/website-status/pull/989#pullrequestreview-1724069975 The issue we are addressing in this task is only to update query params, the comment that you have put is out of scope for this issue. Here is the issue link, acceptance criteria has been mentioned very clearly : https://github.com/Real-Dev-Squad/website-status/issues/971
The search should show closest suggestion of query EX - If user search
Dark mod
missinge
, the result should show fordark mode
currently it is showing the exact value to queryHello, regarding this comment of yours Gaurav : https://github.com/Real-Dev-Squad/website-status/pull/989#pullrequestreview-1724069975 The issue we are addressing in this task is only to update query params, the comment that you have put is out of scope for this issue. Here is the issue link, acceptance criteria has been mentioned very clearly : https://github.com/Real-Dev-Squad/website-status/issues/971
If the behaviour is same, after making your changes then it's fine.
Developer name : @j24m
Backend Changes :
Frontend changes :
Is Under Feature Flag :
Database changes :
Breaking changes :
Tested on local :
Issue :
Description :
Code Changes :
Anything you would like to inform the reviewer about:
Dev Tested :
Test Stats :
Images/video of the change :
https://github.com/Real-Dev-Squad/website-status/assets/53934353/4f88280b-4a3a-4d90-b767-fce913c1aa15
Follow-up Issues (if any) :