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

Dateispeicher Browser #196

Merged
merged 7 commits into from Mar 17, 2024
Merged

Dateispeicher Browser #196

merged 7 commits into from Mar 17, 2024

Conversation

alessioC42
Copy link
Owner

@alessioC42 alessioC42 commented Mar 14, 2024

Add a browser for the "Dateispeicher" Applet based on the existing Parser.

  • basic functionality
  • error handling
  • file search
  • mb show if a file is already in storage (problems with different files and same url?)

@CodeSpoof Do you know if a modified file has a different download URL when it has the same remote path and filename? Also does your school have the search in files feature? Is it just a different AJAX request lanis can disable from school to school like the replacements.

@alessioC42 alessioC42 added the enhancement New feature or request label Mar 14, 2024
@alessioC42 alessioC42 self-assigned this Mar 14, 2024
@alessioC42 alessioC42 changed the title basic datastorage functionality Dateispeicher Browser Mar 14, 2024
@alessioC42 alessioC42 marked this pull request as draft March 14, 2024 19:51
@CodeSpoof
Copy link
Collaborator

Files can only be renamed, not edited. Thus the id will change when reuploading a modified version. The SPH is patrtially inconsistent when it comes to keeping file-IDs when renaming.

@CodeSpoof
Copy link
Collaborator

Search will be tested tomorrow.

@alessioC42
Copy link
Owner Author

Search will be tested tomorrow.

I've implemented the endpoint on the branch. You can test it right there

@alessioC42
Copy link
Owner Author

rec.mp4

@alessioC42 alessioC42 marked this pull request as ready for review March 16, 2024 12:31
@alessioC42 alessioC42 merged commit 398d8bf into main Mar 17, 2024
@alessioC42 alessioC42 deleted the datastorage branch March 17, 2024 11:27
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

Successfully merging this pull request may close these issues.

None yet

2 participants