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.

Sign up for GitHub

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

Jump to bottom

Changed file's name isn't saved when user presses Enter #7324

Closed
BOHEUS opened this issue Sep 29, 2024 · 1 comment · Fixed by #7376
Closed

Changed file's name isn't saved when user presses Enter #7324

BOHEUS opened this issue Sep 29, 2024 · 1 comment · Fixed by #7376
Assignees
@harshit078
Labels
good first issue Good for newcomers prio: low scope: front Issues that are affecting the frontend side only

Comments

@BOHEUS
Copy link
Contributor

BOHEUS commented Sep 29, 2024

Scenario:

  1. Log in
  2. Go to People > record details > (doesn't matter which object as long as it has field with File type)
  3. Upload a file if there are no files
  4. Click on three vertical dots near date of upload and select "Rename" option
  5. Change name of the file and press Enter key

Actual: Changed filename isn't saved after pressing Enter key but only when user clicks outside of the input

Expected: Changed filename must be saved after pressing Enter key or after clicking outside of the input
@harshit078
Copy link
Contributor

@BOHEUS, I'm already working in the file area I'll take this up !

@Bonapara Bonapara added good first issue Good for newcomers scope: front Issues that are affecting the frontend side only prio: low labels Sep 30, 2024
@harshit078 harshit078 mentioned this issue Oct 1, 2024
Merged
lucasbordeau added a commit that referenced this issue Oct 10, 2024
@harshit078 @lucasbordeau
fix: Files field fix (#7376)
54c328a 
## Description

Closes #7324 
Closes #7323

- This PR solves the issue - 
- #7324 
- #7323
- On Enter the rename of the file is saved
- Removed renaming of extension

## After Changes Behaviour


https://github.com/user-attachments/assets/f5c47cd4-883e-473e-9cfa-e277f8f630c2

---------

Co-authored-by: Lucas Bordeau <[email protected]>
harshit078 added a commit to harshit078/twenty that referenced this issue Oct 14, 2024
@harshit078 @lucasbordeau
fix: Files field fix (twentyhq#7376)
5a367ef 
## Description

Closes twentyhq#7324 
Closes twentyhq#7323

- This PR solves the issue - 
- twentyhq#7324 
- twentyhq#7323
- On Enter the rename of the file is saved
- Removed renaming of extension

## After Changes Behaviour


https://github.com/user-attachments/assets/f5c47cd4-883e-473e-9cfa-e277f8f630c2

---------

Co-authored-by: Lucas Bordeau <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@harshit078 harshit078

Labels
good first issue Good for newcomers prio: low scope: front Issues that are affecting the frontend side only
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Files field fix harshit078/twenty
3 participants
@Bonapara @BOHEUS @harshit078