twill icon indicating copy to clipboard operation
twill copied to clipboard

[2.x] Feature: Allow displaying the date of an uploaded file

Open gilbo75 opened this issue 1 year ago • 3 comments

This feature allows displaying the date of an uploaded file in the file library. This option is manageable with a Twill variable.

gilbo75 avatar Mar 08 '23 13:03 gilbo75

PR Summary

  • 📆 New config option for showing upload dates Added a new configuration option to display the uploaded date of files in the media library.
  • 🏞️ MediaSidebar update Made updates to the MediaSidebar component for showcasing this information when enabled.
  • 🌐 Translations for "Uploaded on" Added translations for "Uploaded on" in English and French languages.

what-the-diff[bot] avatar Mar 27 '23 15:03 what-the-diff[bot]

Hello, yes, I need this feature on Twill 2

On Wed, Jun 5, 2024 at 1:55 AM Quentin Renard @.***> wrote:

@.**** commented on this pull request.

In src/Models/File.php https://github.com/area17/twill/pull/2173#discussion_r1626758149:

@@ -43,7 +43,9 @@ public function toCmsArray() 'original' => FileService::getUrl($this->uuid), 'size' => $this->size, 'filesizeInMb' => number_format($this->attributes['size'] / 1048576, 2),

  •    ];
    
  •    ] + (config('twill.file-library.show_uploaded_date')
    
  •        ? ['uploadedDate' => $this->created_at->format("d/m/Y H:i")]
    

@gilbo75 https://github.com/gilbo75 do you need this feature on a Twill 2 project? If you do, please address the issue mentioned by Tofandel and I can merge and release a new minor on Twill 2.

— Reply to this email directly, view it on GitHub https://github.com/area17/twill/pull/2173#discussion_r1626758149, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJXVNP242O6OD2WYXQL6EZLZFZHWRAVCNFSM6AAAAAAVTYTJKKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAOJXGU4TQNRXGU . You are receiving this because you were mentioned.Message ID: @.***>

gilbo75 avatar Jun 05 '24 06:06 gilbo75

@gilbo75 Why did you close the PR instead of just committing the suggested change?

Tofandel avatar Jun 05 '24 16:06 Tofandel