landscapeapp icon indicating copy to clipboard operation
landscapeapp copied to clipboard

addExternalInfo: Correctly reuse cached crunchbase info

Open danielsilverstone-ct opened this issue 3 years ago • 4 comments

When a crunchbase key is not in the environment; but the settings request crunchbase data, we should reuse the data from the processed landscape file.

This fixes #835

danielsilverstone-ct avatar Jul 25 '22 10:07 danielsilverstone-ct

Deploy Preview for landscapeapp ready!

Name Link
Latest commit 957a4e113c5e043818389ff565ef5f0eb150009b
Latest deploy log https://app.netlify.com/sites/landscapeapp/deploys/62de6e6aed5f6e00081fbc02
Deploy Preview https://deploy-preview-836--landscapeapp.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] avatar Jul 25 '22 10:07 netlify[bot]

This works for me.

milanlakhani avatar Jul 25 '22 12:07 milanlakhani

global.skip_crunchbase

AndreyKozlov1984 avatar Aug 10 '22 13:08 AndreyKozlov1984

global.skip_crunchbase

This breaks landscapes which are designed to consume crunchbase data (such as the CNCF one) - it is clearly a bug that the cached crunchbase data was not being used as evidenced by the message: CRUNCHBASE_KEY_4 is not set. Using processed_landscape.yml as a source for crunchbase info which was emitted even though the info was then promptly deleted from the processed landscape.

danielsilverstone-ct avatar Aug 10 '22 14:08 danielsilverstone-ct