-
Notifications
You must be signed in to change notification settings - Fork 362
feat(): Add integration-foot content to destination pages #297
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@sanscontext I would need your approval here as after merging this content will be doubled on pages where its also included in the markdown file It's required to remove include that @bartlomiej-krakowski showing above? Do we also have to do that? Xavier added this lines yesterday and we not sure now |
Approved! We can remove the manually-added includes ourselves - we were doing that in case we couldn't get it working. (All of those empty files were created as a placeholder for new destinations that don't have any docs yet.) |
@@ -1,4 +1,3 @@ | |||
<hr/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aha! Thank you! I couldn't figure out where that was coming from!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One request :)
Closes: #283
Note:
Please check out integrations files, because in some of these is included
integration-foot.md
file.