photo-upload #105

Merged

trentlarson merged 20 commits from photo-upload into master 6 months ago

trentlarson commented 6 months ago

Owner

There is no content yet.

trentlarson added 20 commits 6 months ago

c696de33f3 add page to take a picture and upload to an image server

ca90447700 fix different "environment" variables for prod & dev

92b924643e fix camera resolution, parameterize image API server

f1bb1b51aa Merge branch 'master' into photo-upload

0eb64ed716 add authentication token for image server, change default image server to localhost

cae2bbc4ff make styled button to take picture

acee761906 add page for extended details of gifts including pic (not fully tested)

792e9cb648 separate picture taking from uploading

c58f012d2c allow viewing and deletion of an image

e4543457e2 add image onto give claim, then display on feel (full round-trip, baby!)

c602c5ce50 add some other image deletions in other cases

ae857f4c8f guard against another set of errors when deleting an image

26582030df add another check when deleting an image

476d35452a send the claim type with an image

67f34f9826 on "give details" page, distinguish between project & user destination

49bb1c07b7 fix file extension

251986d2bc make the photo show in a pop-up dialog

252952e017 show the image rate limits

3e556dfa52 move the "part of project" text in giving-details screen

64322b2804 change the default image server port

trentlarson requested review from anomalist 6 months ago

trentlarson requested review from jsnbuchanan 6 months ago

trentlarson self-assigned this 6 months ago

trentlarson merged commit 85ad295eb9 into master 6 months ago

trentlarson referenced this issue from a commit 6 months ago

Merge pull request 'photo-upload' (#105) from photo-upload into master

Reviewers

anomalist was requested for review 6 months ago

jsnbuchanan was requested for review 6 months ago

The pull request has been merged as 85ad295eb9.

Sign in to join this conversation.

No Label

No Milestone

1 Participants

Notifications

Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.