ryanswrt/autoform-slingshot

v1.0.0 release :: Issue #3 :: Deleting saved image does not update DB accordingly

Closed this issue · 8 comments

kyooriouskoala commented

Removing an existing image and submitting the form to save it no longer update the DB accordingly as it would in the previous version.

timbrandin commented

How does the data look before and after saving the form?

kyooriouskoala commented

Schema

picture_url: {
    type: String,
    optional: true,
    autoform: {
        afFieldInput: {
            type: 'slingshotFileUpload',
            slingshotdirective: 'myFileUploads'
        }
    }
}

DB

{
    "_id" : "ZzFBkTncqsGZ2M4xS",
    "product" : "Cookie Monster",
    "createdBy" : "BjoP2iqpHgJyYmjv1",
    "createdAt" : ISODate("2015-07-29T04:55:25.178Z"),
    "editedBy" : "BjoP2iqpHgJyYmjv1",
    "editedAt" : ISODate("2015-08-03T10:10:24.914Z"),
    "picture_url" : "https://folhas.s3-ap-southeast-1.amazonaws.com/best/cookie.jpg"
}

No change to the picture_url key-value when removing the image and saving it.

Replacing an existing image with a new one works fine.

timbrandin commented

Thx, I'll take a look at it. :)

kyooriouskoala commented

timbrandin:autoform-slingshot@1.1.0

Test steps:

  1. Edit existing product entry.
  2. Remove image.
  3. Save.

Result
The picture_url key-value still exists in the collection doc.

timbrandin commented

Ok, I'll look into a little more. How about the other issues?

2015-08-04 12:05 GMT+02:00 kyooriouskoala notifications@github.com:

timbrandin:autoform-slingshot@1.1.0

Test steps:

  1. Edit existing product entry.
  2. Remove image.
  3. Save.

Result
The picture_url key-value still exists in the collection doc.

—
Reply to this email directly or view it on GitHub
#12 (comment)
.

Cheers,
// Tim Brandin @timbrandin https://twitter.com/#!/timbrandin
http://timbrandin.com
http://studiointeract.com

timbrandin commented

This should be resolved in #13, would be great if you could confirm that @kyooriouskoala

kyooriouskoala commented

Hi @timbrandin, this issue is fixed! Thanks :)

timbrandin commented

Perfect!

Skickat från min iPhone

7 aug 2015 kl. 07:10 skrev kyooriouskoala notifications@github.com:

Closed #12.

—
Reply to this email directly or view it on GitHub.