Skip to content

Adding Images#fromBase64 - #310

Merged
erickzanardo merged 3 commits into
developfrom
feat/image-base64
May 5, 2020
Merged

Adding Images#fromBase64#310
erickzanardo merged 3 commits into
developfrom
feat/image-base64

Conversation

@erickzanardo

Copy link
Copy Markdown
Member

Adding a way to load images from base64 string images.

@spydon spydon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, except that generated file, but maybe I just don't understand it?

@@ -0,0 +1,16 @@
//
// Generated file. Do not edit.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this file?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is a new file that all flutter projects have now but these old examples built with older flutter didn't. not sure if we should version or not but it doesn't say on the file it shouldn't be. I would look at the other newer examples to see what we do. maybe the new flutter create adds it to the gitignore.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just checked, it is on git ignore, I am going to remove it

@luanpotter luanpotter left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@erickzanardo
erickzanardo merged commit 52307dd into develop May 5, 2020
@erickzanardo
erickzanardo deleted the feat/image-base64 branch May 5, 2020 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants