wetransfer 0.4.1 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 60a6b133bf3e99a084dd9b480df89119d63d4edb
-  data.tar.gz: 3a8ee6107e57abbeab3e65406a92acaea0c89826
+  metadata.gz: ab826ed8a40904825cc771a81096be1faa229f3d
+  data.tar.gz: bbb35f3de5f2b7f85a6864f33d3d07366270784e
 SHA512:
-  metadata.gz: eb766106e8761c83f2983e5a2661051f486007e33d66933991037e73e48420c48e9bbec28e2513a257cf892c03a8db3e0662fca62f67684a7581cb3376ea80bd
-  data.tar.gz: 82b70497540b93f0d737fb3070e969651f8b27f13e3e12a2ec3237a55bc3f5497dcab182dae74a5fecbdd201e317aa1dcfe34e8ed4debdd2689fdceb11c1be66
+  metadata.gz: e80285c5222256824c55591db0acc9161ce1ad5ab78f84b1c15a165952681c59963261a0bbd23079453b63e73befd4b4b5c8753e2be0239717ae413b1cc3beb4
+  data.tar.gz: 7bdef21c4652477b276058c12add0a0a53167dc2781beceff3f8294f18e021568ee7696fad787ce701e2d2fc395d7e3d659728ea87eb53b9f56cdabb00fa58ae

data/README.md

@@ -68,7 +68,7 @@ transfer = @client.create_transfer(name: "My wonderful transfer", description: "
   upload.add_file_at(path: '/path/to/local/file.jpg')
   upload.add_file_at(path: '/path/to/another/local/file.jpg')
   upload.add_file(name: 'README.txt', io: StringIO.new("This is the contents of the file"))
-  upload.add_web_content(url: "https://www.the.url.you.want.to.share.com"))
+  upload.add_web_url(url: "https://www.the.url.you.want.to.share.com", title: "title of the url"))
 end
 
 transfer.shortened_url => "https://we.tl/SSBsb3ZlIHJ1Ynk="

data/lib/we_transfer_client/transfer_builder.rb

@@ -16,8 +16,9 @@ class TransferBuilder
     add_file(name: File.basename(path), io: File.open(path, 'rb'))
   end
 
-  def add_web_content(url:)
-    @items << FutureWebItem.new(url: url, title: url)
+  def add_web_url(url:, title: nil)
+    title ||= url
+    @items << FutureWebItem.new(url: url, title: title)
     true
   end
 

data/lib/we_transfer_client/version.rb

@@ -1,3 +1,3 @@
 class WeTransferClient
-  VERSION = '0.4.1'
+  VERSION = '0.4.2'
 end

data/spec/integration_spec.rb

@@ -97,7 +97,7 @@ describe WeTransferClient do
     client = WeTransferClient.new(api_key: ENV.fetch('WT_API_KEY'), logger: test_logger)
     transfer = client.create_transfer(name: 'My collection of web content', description: 'link collection') do |builder|
       10.times do
-        builder.add_web_content(url: 'https://www.wetransfer.com')
+        builder.add_web_url(url: 'https://www.wetransfer.com')
       end
     end
     expect(transfer).to be_kind_of(RemoteTransfer)
@@ -135,7 +135,7 @@ describe WeTransferClient do
       expect(add_result).to eq(true)
 
       # add url to transfer
-      add_result = builder.add_web_content(url: 'http://www.wetransfer.com')
+      add_result = builder.add_web_url(url: 'http://www.wetransfer.com', title: 'website used for file transfers')
       expect(add_result).to eq(true)
     end
 

data/spec/we_transfer_client/transfer_builder_spec.rb

@@ -29,12 +29,33 @@ describe TransferBuilder do
 
   it 'should add a url' do
     transfer_builder = described_class.new
-    transfer_builder.add_web_content(url: 'https://www.wetransfer.com/')
+    transfer_builder.add_web_url(url: 'https://www.wetransfer.com/')
     expect(transfer_builder.items.count).to eq(1)
 
     item = transfer_builder.items.first
     expect(item.url).to eq('https://www.wetransfer.com/')
+    expect(item.local_identifier).to be_kind_of(String)
+  end
+
+  it 'should user url for the title when none is given' do
+    transfer_builder = described_class.new
+    transfer_builder.add_web_url(url: 'https://www.wetransfer.com/')
+    expect(transfer_builder.items.count).to eq(1)
+
+    item = transfer_builder.items.first
+
     expect(item.title).to eq('https://www.wetransfer.com/')
     expect(item.local_identifier).to be_kind_of(String)
   end
+
+  it 'should pass title as webcontent title' do
+    transfer_builder = described_class.new
+    transfer_builder.add_web_url(url: 'https://www.wetransfer.com/', title: 'awesome website for file transfering')
+    expect(transfer_builder.items.count).to eq(1)
+
+    item = transfer_builder.items.first
+    expect(item.url).to eq('https://www.wetransfer.com/')
+    expect(item.title).to eq('awesome website for file transfering')
+    expect(item.local_identifier).to be_kind_of(String)
+  end
 end

data/wetransfer.gemspec

@@ -27,11 +27,6 @@ Gem::Specification.new do |spec|
   spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
-  spec.post_install_message = %q{
-    Warning: We have changed the syntax for the create_transfer method.
-    create_transfer(title:, message:) is now create_transfer(name:, description:)
-    Please update your usage accordingly. Thank you and our apologies for the disruption.
-  }
 
   spec.add_dependency 'faraday', '~> 0.12'
   spec.add_dependency 'ks', '~> 0.0.1'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: wetransfer
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.4.2
 platform: ruby
 authors:
 - Noah Berman
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2018-06-27 00:00:00.000000000 Z
+date: 2018-06-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -198,10 +198,7 @@ licenses:
 - MIT
 metadata:
   allowed_push_host: https://rubygems.org
-post_install_message: "\n    Warning: We have changed the syntax for the create_transfer
-  method.\n    create_transfer(title:, message:) is now create_transfer(name:, description:)\n
-  \   Please update your usage accordingly. Thank you and our apologies for the disruption.\n
-  \ "
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib