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

Set a record name on Kanban card creation #6957

Closed
Bonapara opened this issue Sep 9, 2024 · 12 comments · Fixed by #7169
Closed

Set a record name on Kanban card creation #6957

Bonapara opened this issue Sep 9, 2024 · 12 comments · Fixed by #7169
Assignees
Labels
for experienced contributor prio: high scope: front Issues that are affecting the frontend side only size: short

Comments

@Bonapara
Copy link
Member

Bonapara commented Sep 9, 2024

Current behavior

When creating a new record on a kanban board, you cannot set a name for the record from the kanban view. To name the record, you must go to the record detail page.

This doesn't apply to the opportunities kanbans where a specific behavior applies.

CleanShot.2024-09-09.at.17.23.03.mp4

Desired Behavior

We should implement the same behavior we have on table views: when creating a new record, we should display an input to set the record name from the board view. If the input is left empty, the card should be deleted, and the record should not be saved.

This doesn't apply to Opportunities kanbans.

CleanShot 2024-09-09 at 17 34 55

The input label should be the object identifier. We should display the double input for people kanbans.

Table behavior to copy:

CleanShot.2024-09-09.at.17.37.35.mp4
@Faisal-imtiyaz123
Copy link
Contributor

@Bonapara I would like to work.

@Bonapara
Copy link
Member Author

Hi @Faisal-imtiyaz123, thanks! I believe it will be handled along with the other issue!

@Faisal-imtiyaz123
Copy link
Contributor

@Bonapara Sure!

@ehconitin
Copy link
Contributor

@Bonapara
I think Ill work on this.
I still cant figure out #7002 :(

@Faisal-imtiyaz123
Copy link
Contributor

@ehconitin @Bonapara can i work on #7002 since @ehconitin hasn't been able to figure it out yet?

@ehconitin
Copy link
Contributor

@Faisal-imtiyaz123
Please, go ahead.
Let me know if you need any help

@lucasbordeau
Copy link
Contributor

lucasbordeau commented Sep 16, 2024

Please @ehconitin could you drive this issue ? It's more important than #7002 and I'll be really busy with code review this week so I'll appreciate that you focus on this.

@ehconitin
Copy link
Contributor

Got it!

@ehconitin
Copy link
Contributor

ehconitin commented Sep 17, 2024

@lucasbordeau
I am facing the same issue what I faced in #7002
I just cant think of an efficient way to render a card or dropdown in that particular stage or column when clicked on top right add '+', adding them from inside a column or stage is pretty straight forward.
Should we have something similiar to what we have for tables in RecordIndexPage, where we setPendingRecordId(v4()) and then setSelectedTableCellEditMode(-1, 0)
But then I couldnt figure how it would work in context of kanban.
Need help :)

@lucasbordeau
Copy link
Contributor

Yes we should follow the same approach !

@lucasbordeau
Copy link
Contributor

Each column has its own scope so you should find a way to get the scope of a column and then all the recoil component state can be called by passing this scope as a parameter.

We use the parameter precisely for this use case where we want to access the component states from outside of their context.

@ehconitin
Copy link
Contributor

@lucasbordeau Got it! Will let you know if I get stuck somewhere!

@ehconitin ehconitin moved this from 🆕 New to 👀 In review in Product development ✅ Sep 25, 2024
bosiraphael pushed a commit that referenced this issue Sep 25, 2024
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Product development ✅ Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
for experienced contributor prio: high scope: front Issues that are affecting the frontend side only size: short
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants