Real-Dev-Squad / website-status

Shows a roadmap of the ongoing projects being done
https://status.realdevsquad.com/
MIT License
21 stars 145 forks source link

Adding User-name suggestions to AssignedTo field #575

Closed fakhruddinkw closed 11 months ago

fakhruddinkw commented 1 year ago

Closes #560

To be done:

ETA - 25/05/2023

https://github.com/Real-Dev-Squad/website-status/assets/85249044/4d754e2c-6817-417b-b6f5-91d15e18e802

no-user-found

RESPONSIVENESS

https://github.com/Real-Dev-Squad/website-status/assets/85249044/0cc2716e-d87b-4c5f-a5c7-22d755bbe1c8

EDIT WILL BE ONLY AVAILABLE IF EDIT=TRUE AND USER=SUPER_USER

https://github.com/Real-Dev-Squad/website-status/assets/85249044/7b896703-6347-4db2-9ab6-c659eed0544a

https://github.com/Real-Dev-Squad/website-status/assets/85249044/093382c9-ed70-456c-a2d9-e22134c66190

It takes time for changes to reflect as you can see in the second video. But the changes are made successfully, as indicated by the success message in the first video

vercel[bot] commented 1 year 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 Jun 10, 2023 1:28pm
shubhamsinghbundela commented 11 months ago

Hi @fakhruddinkw Can you please add before and after image/video in description For more visualization of UI

shubhamsinghbundela commented 11 months ago

Hello @fakhruddinkw, I get confused after seeing the video When super_user press Alt then input field appear and he can type the name on input field

and after typing the name and user get selected and the input field disappear and username get updated and we can see like before i.e. like this image

image

Correct me If I am wrong??

cc: @bhtibrewal @Pratiyushkumar @vvaibhavdesai @ankushdharkar @RitikJaiswal75

fakhruddinkw commented 11 months ago

Hi @fakhruddinkw great work...I written some comment can you please reply to those comment and also where you written test??

Shubham tests are ready to go. I just need this PR to be merge, so i can push it, as this PR got bigger i am keeping it separate.

fakhruddinkw commented 11 months ago

Left few small suggestions, please look into them, in okay you can address them in a follow-up

Thank you @bhtibrewal ! I will clean this up when i push the tests in next PR. Thank you again for understanding 🙏