kestra-io / kestra

:zap: Workflow Automation Platform. Orchestrate & Schedule code in any language, run anywhere, 500+ plugins. Alternative to Zapier, Rundeck, Camunda, Airflow...
https://kestra.io
Apache License 2.0
10.58k stars 901 forks source link

Ability to preview jpeg png files in NamespaceFiles #4490

Open smunteankestra opened 2 months ago

smunteankestra commented 2 months ago

Feature description

Would be great if kestra would rpeview Jpeg or PNG files currently we have this

image

SyedMDot commented 1 week ago

Hi, I want to contribute to this issue. could you please assign this to me? @anna-geller

SyedMDot commented 6 days ago

Thank you for assigning, I'll try my best to sort it out!

Ayush-Dhawan commented 4 days ago

Can I try to work on the issue?

MilosPaunovic commented 4 days ago

We can check if @SyedMDot is already working on it, so let's wait for the answer. In the meantime, you can our list of filtered issues with the good first issue label and take your pick at something else.

Also, to build the project, you can find some guides here: https://kestra.io/docs/getting-started/contributing

SyedMDot commented 4 days ago

We can check if @SyedMDot is already working on it, so let's wait for the answer. In the meantime, you can our list of filtered issues with the good first issue label and take your pick at something else.

Also, to build the project, you can find some guides here: https://kestra.io/docs/getting-started/contributing

Hi @MilosPaunovic I had tried the best and go deep down into this issue, I debug relevant code, but I only find one issue, i.e. on creation of file, in the namespace.js file, it is taking "text/plain" as a mime type, that's why only txt files are readable, so I modified that method according to the type of file imported.

But it isn't working as intended, you can assign this to anyone else, meanwhile I'm looking into this.

I will be glad to contribute if I figure out something... else I'll learn something new by observing others contributions.

Thank you!

MilosPaunovic commented 4 days ago

Then it means you can try and tackle this one @Ayush-Dhawan.

Thanks for the invested effort @SyedMDot! 🚀