show asset_store_key and asset_metadata in Definition's Outputs section
wrapped it in AssetStoreContainer component as we will probably add more info there as the asset land grows
This mostly lgtm. I think it looks a little weird to have "null" when there's no asset metadata. What do you think about just leaving it blank instead for that situation?
OutputDefinition now just has asset_store_key and asset_metadata, so we should probably do that here too?