-
-
Notifications
You must be signed in to change notification settings - Fork 418
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
Image Uploads #287
Open
atomdmac
wants to merge
26
commits into
lovasoa:master
Choose a base branch
from
atomdmac:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Image Uploads #287
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
714c73e
Add library multiparty to parse form data.
b166eb8
Use BoardDataList instead of plain object
8fa13cb
Return public interface for Sockets object
073ff19
Image asset uploads and updates.
b23bf11
Validate image size before allowing upload.
7abbda4
Get MIME type before returning image assets.
ff2768f
Remove old TODO
f278e68
Use XHR to upload images. Allows access to progress.
6b847cf
Move all image upload stuff to image tool
bf67be1
Use image tool to initiate upload
15781ee
Delete images if they are no longer in use.
f1cd94d
Drop onto board, not canvas.
daed5e3
Remove unnecessary console.log
962bb22
Use cell cursor for image tool.
9bf0e3f
Disable image upload if Image tool is in block list
5d82a78
Remove tmp file after image upload.
d09411b
Don't block main handler fn when serving board image assets.
84ff1e1
Validate image uploads on client and server.
1a815a6
Add log for when unused images are purged
c321964
Ensure that board image asset URLs always point to current domain.
1e3708e
Use fs.promises to read board image assets.
7fe1fe3
Use board name and asset ID to build image href
b0e47a1
Fix image purge.
231d3fd
Attach click and drag/drop handlers to canvas, not drawingArea
dfd96b1
Add tests for image upload tool.
05d0ea2
Merge branch 'master' into master
atomdmac File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is complicated. Can we remove the src altogether ?
that would stress me less :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How would the browser know where to pull the image to display from?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The browser would make a request to
boards/{boardname}/images/{image_id}
, the server would check that the image exists, and if so, serve it.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yours is a much better approach 😅 This has been updated.