Skip to content
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

Changes from AngularDocFX_EN #1199

Merged
merged 134 commits into from
May 29, 2019
Merged

Conversation

ESShared
Copy link
Contributor

No description provided.

@zdrawku
Copy link
Contributor

zdrawku commented Apr 18, 2019

@HUSSAR-mtrela and @agoldenbaum we've added some changes to this PR, and a lot more are yet to come, related to issues #1201

My question is, could you show us how (if possible) to create separate PRs with the changes, for better consistency and easing the Verify&Review process? One of the changes will affect more than 300 files, part of the logic that we've added for the Themes in DocFX (all DV topics won't have any theming)

mpavlinov and others added 25 commits May 8, 2019 18:38
change the frame id

Co-Authored-By: Zdravko Kolev <[email protected]>
Update the correct data-iframe-id

Co-Authored-By: Zdravko Kolev <[email protected]>
@pamelabrasil
Copy link
Contributor

@zdrawku @HUSSAR-mtrela Some of the changes in this PR revert a couple of changes from these ones: #1126 #1125 #1120.
The changes are not critical and easily could be applied again through the xplat-docfx, but we should consider how we can even the igniteui-docfx and xplat-docfx versions to prevent future issues

All changes should be done in the xplat-docfx repo. Otherwise, all changes in docfx repo will be overwritten every time there is a PR

@pamelabrasil pamelabrasil dismissed HristoP96’s stale review May 29, 2019 15:38

all changes must be done in the xplat repo. otherwise everything will be overwritten through the PRs

@pamelabrasil pamelabrasil self-requested a review May 29, 2019 15:39
@pamelabrasil pamelabrasil merged commit 3df0b17 into IgniteUI:master May 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.