r7-labs / R7.Documents

Classic DNN Documents module, redesigned
MIT License
20 stars 7 forks source link

can it be set to pull documents from the file manager without uploading via the module itself? #51

Closed pwegrzy closed 5 years ago

pwegrzy commented 7 years ago

Hi, I'm a little confused, If I have a folder in the file manager (DNN 8.0.4), and I have files in that folder that the user that is logged into DNN can view, if I drop this module on a page, and point it to that folder within the file system, shouldn't it show the files by default?

It seems like unless you upload the files via this module, even if you point it to a folder with lots of files in it the user should be able to see, they don't see them???? I have even tried syncing the folder in question (and the recursive sync as well), but it still doesn't show files unless they have been uploaded (or manually mapped) via this tool;s interface.

I already have tons of files in the site file system, and I am just looking to expose those with a lightweight file viewer.

thanks, otherwise, the module seems to work as expected.

roman-yagodin commented 7 years ago

Thanks for your interest, @pwegrzy! Currently, if you specify default folder in module settings, the module will try to select that folder by default in the "Add new document" UI, which gives somewhat faster results then adding several new documents in a row.

Some features alike are planned (#44), but currently I'm unsure about exact kind of implementation. If could be ability to display the list of files from specified folder (it looks like your case) or ability to bulk add new documents from a folder, or maybe both.

RichPruitt commented 7 years ago

I'd really like to have the ability to point it to the file system folder and have it sync the documents shown in the module with the folder.

Use Case: I have a folder that we drop our nightly builds on and would like for our registered users to be able to get that file as needed. I dont want to have to log in every day and add it manually to the document list.

BTW Kudos on the tool, much better than the standard documents module.

roman-yagodin commented 7 years ago

Thanks for your interest!

I'd like to clarify some things - e.g. is it OK for you to have filename as document title, or custom title is required? Which fields you would want to have display in document list in the use case you've described?

RichPruitt commented 7 years ago

I think the file name would be fine as the document title. In this use case just the file name and date created and a way to download would be fine. Thanks!

johnachisholm commented 6 years ago

I would also like (if the module does not automatically add the contents of a specified directory), that once a file is added to the module, it will check the file system and send the file. I, similar to RichPruitt, have nightly .PDF's that are updated by an automated process (they overwrite a file of the same name). I would like the system to automatically send the most recent copy of the file that exists on the file system (and ideally, update the modified date that the module shows to the timestamp on the file).

This is a great module. Thank you for your work on it!

roman-yagodin commented 6 years ago

https://github.com/roman-yagodin/R7.Documents/tree/feature-51

johnachisholm commented 6 years ago

Hi –

Thank you. For some reason, I am having problems building the solution. It seems to have multiple parsing errors. Are you able to offer any ideas on why this would be?

[cid:image006.jpg@01D3F761.103BCFC0]

John A. Chisholm Captain / EMS Officer Concord Fire Department 24 Horseshoe Pond Lane Concord, NH 03301 Phone: (603) 225-8650

New Hampshire’s Main Street™ [cid:image001.png@01D202A4.7E0ED8C0]

From: Roman M. Yagodin [mailto:notifications@github.com] Sent: Tuesday, May 29, 2018 2:13 AM To: roman-yagodin/R7.Documents Cc: Chisholm, John; Comment Subject: Re: [roman-yagodin/R7.Documents] can it be set to pull documents from the file manager without uploading via the module itself? (#51)

https://github.com/roman-yagodin/R7.Documents/tree/feature-51

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/roman-yagodin/R7.Documents/issues/51#issuecomment-392663477, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZWiqkM0j3ER-UYoR80wpbb5Q-PReVzIks5t3ObwgaJpZM4LyFh6.

roman-yagodin commented 6 years ago

I currently develop using MonoDevelop 7.4 and Visual Studio Code (on Linux, you know). So if you try to load solution in the Visual Studio, some minor quirks are the things that should be expected. But msbuild /p:Configuration=Release R7.Documents.sln shouldn't have problems with that.

Never mind, here is a pretty raw development build you can test: R7.Documents-01.10.00-Install.Release.zip Edit: See https://github.com/roman-yagodin/R7.Documents/releases/tag/1.11.0-beta.1 Don't use it in production, you have been warned!

johnachisholm commented 6 years ago

Thank you. I found the issue. visual studio 2015 needed a package update because of some newer C#6 syntax that was used. The compiler didn’t like it until I updated the Msbuild tools.

John

On May 31, 2018, at 3:30 AM, Roman M. Yagodin notifications@github.com<mailto:notifications@github.com> wrote:

I generally develop using MonoDevelop 7.4 and Visual Studio Code (on Linux, you know). So if you try to load solution in the Visual Studio, some minor quirks are that should be expected. But msbuild /p:Configuration=Release R7.Documents.sln shouldn't have problems with that.

Never mind, here is a raw development build you can test: R7.Documents-01.10.00-Install.Release.ziphttps://github.com/roman-yagodin/R7.Documents/files/2057203/R7.Documents-01.10.00-Install.Release.zip Don't use it in production, you have been warned!

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/roman-yagodin/R7.Documents/issues/51#issuecomment-393436966, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZWiqsYNrdCBrLXAZn13FlFOJDTVAeILks5t35wsgaJpZM4LyFh6.

roman-yagodin commented 6 years ago

https://github.com/roman-yagodin/R7.Documents/releases/tag/1.11.0-beta.1