CartoDB / observatory-extension

BSD 3-Clause "New" or "Revised" License
6 stars 4 forks source link

modified license content as per Operations request #125

Closed csobier closed 8 years ago

csobier commented 8 years ago

For Docs issue#879

@vhamer , please approve.

@talos , I've updated the license content as described in the attached issue. The DO PDF link contains a license page that is slightly different then the public docs page. I just want to make sure we're using one golden source for the content.

talos commented 8 years ago

I've updated the license text on the PDF, which should move to link removed .

In the future, changes to the DO catalog should be filed with github.com/cartodb/bigmetadata, since the PDF is generated as part of the ETL/metadata, not as part of the extension.

@csobier I filed a ticket on cartodb-platform (https://github.com/CartoDB/cartodb-platform/issues/2071) to move the catalog PDF, should that be filed with cartodb-docs instead? Thanks!

talos commented 8 years ago

Ah, I see the confusion -- we have a docs page here that isn't really exposed anywhere. I'd say just open issues on the bigmetadata repo and don't worry so much about the LICENSE here, since it doesn't surface anywhere. Maybe we should delete it?

csobier commented 8 years ago

Actually, the license.md file (as shown in Doc/API.md) shows the list of files from this repo that I am using to generate the live docs), so it is being used for the public docs page.

talos commented 8 years ago

Learn something new every day! Then I guess we should just keep tickets on here and have you tag me. Hopefully we don't have to change this text too much.

csobier commented 8 years ago

V. is out all day, I'm making the executive decision to approve this merge, since I used her text verbatim.

Need someone from OBS team to merge this Dev branch into Master?

vhamer commented 8 years ago

Thank you Christa!

On Jun 28, 2016, at 12:22 PM, csobier notifications@github.com wrote:

Merged #125.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.