Skip to content
This repository has been archived by the owner on Oct 20, 2022. It is now read-only.

Add metadata handling #317

Closed
wants to merge 3 commits into from

Conversation

fstau
Copy link
Contributor

@fstau fstau commented May 13, 2022

@agnieszka-m
Implements #306

Difference to spec suggested in issue:
I opted for one comment per attribute vs. my original suggestion. This is more straightforward to parse esp. when having the fields as optionals.

@vercel
Copy link

vercel bot commented May 13, 2022

@fstau is attempting to deploy a commit to the deepset Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented May 18, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
haystack-website ✅ Ready (Inspect) Visit Preview May 18, 2022 at 11:01AM (UTC)

@TuanaCelik
Copy link
Contributor

Hey @fstau - As a final step before merging this PR is it possible to add a dummy section somewhere in the docs with those meta fields as comments on the top so we can test it in the Vercel Preview? If all good we can remove it and merge.

@fstau
Copy link
Contributor Author

fstau commented May 30, 2022

Hey @fstau - As a final step before merging this PR is it possible to add a dummy section somewhere in the docs with those meta fields as comments on the top so we can test it in the Vercel Preview? If all good we can remove it and merge.

Hey @TuanaCelik I have added a sample to the getting started page in the overview group.

@masci
Copy link
Contributor

masci commented Sep 12, 2022

Closing as we're organising the docs elsewhere

@masci masci closed this Sep 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow for metadata population for each webpage
3 participants