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

DEEP LINKING: deepLinkPath property is using the deepest-level name as path and is not modifiable #217

Closed
reganwolfrom opened this issue Apr 13, 2024 · 3 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@reganwolfrom
Copy link
Member

reganwolfrom commented Apr 13, 2024

For deep links, using the deepest-level name may work in some cases, such as SharePoint (?), but for Google Drive, that name is an ID, rather than a human-readable name.

It isn't essential to retrieve a human-readable name, but ideally, it should be possible to provide a name for the 'path' property.

If it is not workable to do so, it may make sense to create a system attribute or property for this deepLinkName value, which could then be used for display when available.

@reganwolfrom reganwolfrom added the enhancement New feature or request label Apr 13, 2024
@reganwolfrom reganwolfrom added this to the v1.15.0 milestone Apr 13, 2024
@mfriesen
Copy link
Member

@reganwolfrom Can you provide examples.

@reganwolfrom
Copy link
Member Author

@mfriesen Here is an example:

{"deepLinkPath":"https://docs.google.com/document/d/1YZWn84REBdKPzlGbirnQOf7qneJgvmDweXWTQeT12O0","lastModifiedDate":"2024-02-29T14:54:01+0000","userId":"87ed86db-058a-4ebb-819a-63b3b36dd61e","path":"1YZWn84REBdKPzlGbirnQOf7qneJgvmDweXWTQeT12O0","insertedDate":"2023-12-15T22:19:40+0000","checksum":"d187f02890b4fcae7cf74d866dcb2d9b","siteId":"default","contentLength":2349320,"documentId":"b83e4b86-0bb3-4580-808e-77a06a01333b","contentType":"application/pdf"}

Ideally, this wouldn't show as a file named 1YZWn84REBdKPzlGbirnQOf7qneJgvmDweXWTQeT12O0.

Also, the contentType is incorrect.

@reganwolfrom
Copy link
Member Author

The plan is to allow path to be modifiable, allowing for file names.

mfriesen added a commit that referenced this issue Apr 19, 2024
mfriesen added a commit that referenced this issue Apr 23, 2024
mfriesen added a commit that referenced this issue May 13, 2024
mfriesen added a commit that referenced this issue May 13, 2024
mfriesen added a commit that referenced this issue May 24, 2024
mfriesen added a commit that referenced this issue Jul 7, 2024
mfriesen added a commit that referenced this issue Jul 7, 2024
* 215 - Add "ocrNumberOfPages" to OCR action

* #207 - GET /documents/{documentId}/content - NoSuchKeyException

* #217 - deepLinkPath property is using the deepest-level name as path and is not modifiable

* #218 - POST /search tag range not inclusive of "end" value

* #228 - add IDP Mapping

* #227 - Modify "Only available with Pro/Enterprise" text in /documents/{id}/ocr endpoints

* #226 - Add ability to publish document

* Updated console to 3.6.0

* #232 - Add Schema Classification

* #231 - Require valid URL on deepLinkPath property

* #195 - Users API
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants