Fix upload_attachments: specify file name in parameter dict

[raph-topo]

Rationale: path on disk may not exist or may not be significant (as with NamedTemporaryFile()).

[ricpol]

Non sure about this - a NamedTemporaryFile is just a file that happens to be managed by tempfile... but you may still get its path with "f.name".
It would be different if you wanted to stream the attachment upload, like this - this, however, involves calling the Requests api with a "data" parameter, which we don't use at the moment.
Support for streaming may be useful though - don't know if the underlying Grist api allows this. I'll add this to my todo list.

or - maybe I didn't understand what you are trying to do here? What use case do you have in mind?

[raph-topo]

Non sure about this - a NamedTemporaryFile is just a file that happens to be managed by tempfile... but you may still get its path with "f.name".

Yes, a random name, which it makes no sense to store in Grist. (But Pygrister download_attachment & upload_attachments seem to need NamedTemporaryFile instead of TemporaryFile if I recall my first tests.)

or - maybe I didn't understand what you are trying to do here? What use case do you have in mind?

I'm using my pygrister fork including this PR to copy attachments from one Grist doc to another, without loosing their name:

        ...
        for att in fr.attachments:
            with NamedTemporaryFile() as file:
                fr.grist.download_attachment(
                    filename=Path(file.name),
                    attachment_id=att.id,
                )
                to_id: int = to.grist.upload_attachments(
                    {att.fields.fileName: Path(file.name)}
                )[1][0]
                ...

(fr.grist & to.grist are instances pygrister.api.GristApi with .open_session() to one Grist doc each)

NamedTemporaryFile has the huge advantage of cleaning itself up when the with block ends. But without the PR, upload_attachments() would store and have Grist display that random name.