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

fix: Save a new paper from the Cozy filePicker #674

Merged
merged 1 commit into from
Oct 15, 2024
Merged

Conversation

Merkur39
Copy link
Member

We also retrieve the original file name, to display something consistent.

### 🐛 Bug Fixes

* Save a new paper from the Cozy filePicker

We also retrieve the original file name,
to display something consistent.
const { data: fileSelected } = await client.query(
buildedFileQueryById.definition,
buildedFileQueryById.options
)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on ne gère pas le loading ici et si je comprends bien le fetch se fait après un click sur un bouton (quand on souhaite sélectionner un fichier), ce n'est pas gênant avec une connexion lente ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ici le fichier sera déjà dans le store puisque sélectionné au préalable. Donc pas de soucis lié au réseaux 👍

@Merkur39 Merkur39 merged commit b54171f into master Oct 15, 2024
1 check passed
@Merkur39 Merkur39 deleted the feat/ver-983 branch October 15, 2024 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants