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

Time of file/folder-modification #320

Open
frepke opened this issue Feb 1, 2021 · 8 comments
Open

Time of file/folder-modification #320

frepke opened this issue Feb 1, 2021 · 8 comments

Comments

@frepke
Copy link

frepke commented Feb 1, 2021

  • Version (cloudcmd -v): 15.5.1
  • Node Version node -v: -
  • OS (uname -a on Linux): Docker version 20.10.2 on Linux PVE 5.4.78-2
  • Browser name/version: Safari/Firefox (all latest)
  • Used Command Line Parameters: -
  • Changed Config: -

I would like to see an option NAME-SIZE-DATE were DATE includes the "modify time".

For some reason I've to reload the webpage sometimes, otherwise I'm working in a file which isn't saved to the disk.
When a modification time is visible, I know if a reload before editing is necessary.

@coderaiser
Copy link
Owner

This is a very good idea for a pull request :). readify is a great place to start from.

@frepke
Copy link
Author

frepke commented Feb 1, 2021

What to do with readify? Do I have to install it in the dockercontainer?

@coderaiser
Copy link
Owner

coderaiser commented Feb 1, 2021

Right now readify returns data in this way:

({
    path: '/',
    files: [{
        name: 'readify.js',
        size: '4.22kb',
        date: '20.02.2016',
        owner: 'coderaiser',
        mode: 'rw- rw- r--',
        type: 'file',
    }],
});

It should be modified to have a time field:

({
    path: '/',
    files: [{
        name: 'readify.js',
        size: '4.22kb',
        date: '20.02.2016',
        time: '21:35',
        owner: 'coderaiser',
        mode: 'rw- rw- r--',
        type: 'file',
    }],
});

Then file template should be modified and name-size-date-time styles should be added to columns directory.

@frepke
Copy link
Author

frepke commented Feb 2, 2021

Unfortunately I don't know where to start, maybe someone else can jump in.

@coderaiser
Copy link
Owner

We can start from determining do you want date and time both was visible or only one of them?
Showing one of them would be much simpler :).

Also what format of time do you want to see? Is 21:35 is ok to you? or you see it as 21:35:59?

@frepke
Copy link
Author

frepke commented Feb 2, 2021

I would prefer both date and time , 21:35 without seconds is okay

@coderaiser
Copy link
Owner

coderaiser commented Feb 2, 2021

In this case addTime function can be added to @cloudcmd/formatify.

And later @cloudcmd/sortify should be updated to add support of sorting by time.

@frepke
Copy link
Author

frepke commented Feb 4, 2021

Sorry, no clue how to implement this.

maybe someone else can pick this up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants