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

chore(release): Update im-metadata to v3.0.0 #55

Closed
wants to merge 48 commits into from

Conversation

anthonyringoet
Copy link
Contributor

No description provided.

@anthonyringoet
Copy link
Contributor Author

Bump im-metadata to fix #10

@anthonyringoet
Copy link
Contributor Author

Need to update the size test to return bytes

@Starefossen
Copy link
Contributor

You got a merge commit in this branch. Can you do a rebase on master:

git reset --hard HEAD~
git rebase origin/master
git push -f

A tip when creating new branches is to branch from the remote branch by git checkout origin/master -b feature/awesome-new-feature 😄

Hans Kristian Flaatten and others added 22 commits October 28, 2015 08:06
Add support for `Expires` and `Caceh-Control` headers for
images uploaded to AWS S3. It is good practice to cache as
much as possible and adding the headers retrospectively to
AWS is kind of a pain.

PR-URL: #45

Signed-off-by: Hans Kristian Flaatten <[email protected]>
Add support for selecting a custom random path generator function such as
`broofa/node-uuid`.

PR-URL: #41
Implements: #20

Signed-off-by: Hans Kristian Flaatten <[email protected]>
PR-URL: #48

Signed-off-by: Hans Kristian Flaatten <[email protected]>
Set absolute path for uploaded image. This disables the random path.
The final path to a version image follows the following rule:

  prefix + path + suffix + ext

PR-URL: #47

Close: #36 #38

Related: #46

Signed-off-by: Hans Kristian Flaatten <[email protected]>
@anthonyringoet
Copy link
Contributor Author

Thanks for the tip, i'll go for a new pr.

@anthonyringoet anthonyringoet deleted the deps/im-metadata-3.0.0 branch October 28, 2015 07:16
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.

4 participants