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

[Bug] Comment Multiline Input Field state isn't getting reset once the comment has been Send #41

Open
Nabhag8848 opened this issue Jan 21, 2024 · 14 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@Nabhag8848
Copy link
Collaborator

What happened?

  • Comment Multiline Input Field state isn't getting reset once the comment has been Send. it should reset the field to empty string.

Steps to reproduce

  1. run '/notion comment'
  2. select page and comment on page.
  3. see the comment input field.

Relevant ScreenShots

Screen.Recording.2024-01-21.at.09.39.02.mov

Version

v6.5

What browsers are you seeing the problem on?

Chrome

@Nabhag8848 Nabhag8848 added the bug Something isn't working label Jan 21, 2024
@brf153
Copy link
Contributor

brf153 commented Jan 22, 2024

@Nabhag8848 capabilities of my integration are
Screenshot from 2024-01-22 21-48-31

Still, I am not being able to make comments on a page.
Screenshot from 2024-01-22 21-49-45

@Nabhag8848
Copy link
Collaborator Author

Nabhag8848 commented Jan 22, 2024

@Nabhag8848 capabilities of my integration are
Screenshot from 2024-01-22 21-48-31

Still, I am not being able to make comments on a page.
Screenshot from 2024-01-22 21-49-45

connect again maybe you didn't have capabilities when you connected and please please please make sure you are working on one issue at a time.

@brf153
Copy link
Contributor

brf153 commented Jan 22, 2024

You mean that I should refresh the page or should I use the command of rc-apps deploy url username password again?

@brf153
Copy link
Contributor

brf153 commented Jan 22, 2024

Yeah I am working on the pagination one. I have made the recursive functions. I just need to check whether they are working fine or not. This issue seems easier than that one so I thought of making a pr for this one first.

@Nabhag8848
Copy link
Collaborator Author

Yeah I am working on the pagination one. I have made the recursive functions. I just need to check whether they are working fine or not. This issue seems easier than that one so I thought of making a pr for this one first.

In my opinion, prioritise that first and come up with this later, this will make things easier for us for reviewing.

@Nabhag8848
Copy link
Collaborator Author

You mean that I should refresh the page or should I use the command of rc-apps deploy url username password again?

connect again /notion connectmaybe you didn't have capabilities when you connected and please please please make sure you are working on one issue at a time.

@brf153
Copy link
Contributor

brf153 commented Jan 22, 2024

@Nabhag8848 oki

@brf153
Copy link
Contributor

brf153 commented Jan 22, 2024

I will finish the previous one first

@VipinDevelops
Copy link

Hello, @Nabhag8848, I'm interested in working on this issue.

@brf153
Copy link
Contributor

brf153 commented Jan 27, 2024

@VipinDevelops I was actually working on this issue

@Sayan4444
Copy link
Contributor

Hi @Nabhag8848 when we click submit,the input text should be equals=''. This feature is already implemented but its only clearing the input field value from the server and not from the client i.e. after sending a message you cant send the same message again as you cant send a emtpy message. I demonstrated it in the below video.

2024-01-29.00-36-53.mp4

If there is a function or a way in which we can also update the client side when the its server side value is changed i.e. we force the input field controlling it from the server, then it should work

@Nabhag8848
Copy link
Collaborator Author

Nabhag8848 commented Jan 28, 2024

Hi @Nabhag8848 when we click submit,the input text should be equals=''. This feature is already implemented but its only clearing the input field value from the server and not from the client i.e. after sending a message you cant send the same message again as you cant send a emtpy message. I demonstrated it in the below video.

2024-01-29.00-36-53.mp4
If there is a function or a way in which we can also update the client side when the its server side value is changed i.e. we force the input field controlling it from the server, then it should work

@Sayan4444 Yep. you got the point. #43 this issue is affecting this too. we have viewErrors everywhere but currently for some reason its not working. handleCommentInputAction is the custom method which is having the logic where we are storing the state on dispatch action of keyboard click when the input comment is written. try to get where this is called and most probably the state isn't resetting when comment is made.

Edited: Check i believe there is an issue around intialvalue of this comment Multiline input element

@Sayan4444
Copy link
Contributor

Sayan4444 commented Jan 28, 2024

@Nabhag8848 handleCommentInputAction is called for every keystrokes we make but we want to this delete the input field text for submit. Moreover input field state can be reset anywhere using this

await modalInteraction.clearInputElementState(
CommentPage.COMMENT_INPUT_ACTION
);

2024-01-29.01-19-30.mp4

I hope my explaination is good enough and do tell me if I lack any understanding

@Nabhag8848
Copy link
Collaborator Author

@Nabhag8848 handleCommentInputAction is called for every keystrokes we make but we want to this delete the input field text for submit. Moreover input field state can be reset anywhere using this

await modalInteraction.clearInputElementState( CommentPage.COMMENT_INPUT_ACTION );

2024-01-29.01-19-30.mp4
I hope my explaination is good enough and do tell me if I lack any understanding

Yep but i believe we are clearlying the input on action of comment, issue lies somewhere else.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
Development

No branches or pull requests

4 participants