Skip to content

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

Merged
merged 2 commits into from
Nov 22, 2019

Conversation

bart-krakowski
Copy link
Contributor

@bart-krakowski bart-krakowski commented Nov 22, 2019

Closes: #283

Note:
Please check out integrations files, because in some of these is included integration-foot.md file.

Screenshot 2019-11-22 at 12 53 20

@jedrzejchalubek
Copy link
Contributor

jedrzejchalubek commented Nov 22, 2019

@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

@sanscontext
Copy link
Contributor

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/>
Copy link
Contributor

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!

Copy link
Contributor

@sanscontext sanscontext left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One request :)

@bart-krakowski bart-krakowski merged commit 575826f into master Nov 22, 2019
@jedrzejchalubek jedrzejchalubek deleted the 283-integration-foot-destinations branch November 28, 2019 10:33
bot-docsteam added a commit that referenced this pull request Apr 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Destination pages should all include integration-foot content
4 participants