goinvo / staffplan-next-app

Next App for Staffplan
https://staffplan-next-app.vercel.app
1 stars 1 forks source link

Add People not working (on Project) #109

Closed jsonin closed 4 days ago

jsonin commented 1 week ago
image
zhukovdigital commented 1 week ago

@jsonin should we used present modal for adding person when we click on header " + People" btn?

zhukovdigital commented 1 week ago

@jsonin or just adding it inline photo_2024-09-06_16-01-00 And this input should be like autocomplete whith present people and TBD?

jsonin commented 1 week ago

When you click on + People, on a particular project staffplan, that new person row (and corresponding open person field) should appear... so the row appears (like in your screenshot). Then I can pick someone who is in our company.

For now, we are catering to small companies, so a pick list/type ahead is fine.

zhukovdigital commented 1 week ago

image @jsonin I need some clarification for correctly behaviour. So, we click on the + People btn -> Show the last row of the table on the screen, in which we have select input, where we can select the user from the present user list, when user select someone it should be like a regular row in the table, correct? And what should be if user doesn`t select anyone from the users list and start to enter new user name in Add person textfield, we should create a new user?

jsonin commented 1 week ago

Oooh, you basically have it right there with TBD. Let's add that TBD row, with the 2 input rows... that looks good. Then people can assign the person through that TBD dropdown menu. Does that work?

zhukovdigital commented 1 week ago

@jsonin yup, will do it and what about the behaviour of the input below "Add Person"?

jsonin commented 1 week ago

I wonder... do we use TEMPORARILY use the TBD row as the "add new"... but we have the label not be TBD but "Add Person"?

So the TBD is an actual choice, like choosing a name... then "Add Person" is the label of the pulldown menu.

zhukovdigital commented 1 week ago

@jsonin, will work on it and then will show you here for confirmation.

zhukovdigital commented 6 days ago

https://github.com/user-attachments/assets/2afc244a-260b-4f65-b976-95dfbbcc805b

@jsonin can u look at this?

zhukovdigital commented 6 days ago

@jsonin is it as expected behaviour?

jsonin commented 6 days ago

Is the "+ People" spawning the TBD and the Add Person input box? As long as someone clicks the +People, and a single new row appears = good.

Having TBD and an open field = not quite right. 1) Click "+People" 2a) one option: just have the TBD full row appear.... that then allows me to switch TBD to a person in the org (with a drop-down menu) or 2b) show the Add Person open input box and the proposed hours input boxes too, then I can look-up the person..

zhukovdigital commented 6 days ago

@jsonin clicked on the +People => show last row on the video, is it correct variant? https://github.com/user-attachments/assets/6f705ef6-a305-4554-a482-833595bef527

jsonin commented 5 days ago

That's spot on. TY!