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

Update documentation for Feathr UI, registry, and architecture. #534

Merged
merged 23 commits into from
Aug 10, 2022

Conversation

xiaoyongzhu
Copy link
Member

@xiaoyongzhu xiaoyongzhu commented Aug 1, 2022

This PR focus on:

Solved doc issue:

#493
#292

@xiaoyongzhu xiaoyongzhu marked this pull request as draft August 1, 2022 07:41
@xiaoyongzhu xiaoyongzhu marked this pull request as ready for review August 9, 2022 09:19
docs/concepts/feature-registry.md Outdated Show resolved Hide resolved
docs/concepts/feature-registry.md Outdated Show resolved Hide resolved
docs/concepts/feature-registry.md Outdated Show resolved Hide resolved
docs/concepts/feature-registry.md Outdated Show resolved Hide resolved
docs/how-to-guides/azure-deployment-arm.md Outdated Show resolved Hide resolved
docs/quickstart_synapse.md Show resolved Hide resolved
Yuqing-cat
Yuqing-cat previously approved these changes Aug 9, 2022
blrchen
blrchen previously approved these changes Aug 10, 2022
Copy link
Collaborator

@blrchen blrchen left a comment

Choose a reason for hiding this comment

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

Thanks for the doc improvements, PR looks good to me, only one minor comment

docs/concepts/feature-registry.md Show resolved Hide resolved
@xiaoyongzhu xiaoyongzhu merged commit 235d9d8 into main Aug 10, 2022
@xiaoyongzhu xiaoyongzhu deleted the xiaoyzhu/release_update_and_registry branch August 10, 2022 05:44
@xiaoyongzhu xiaoyongzhu restored the xiaoyzhu/release_update_and_registry branch August 10, 2022 05:47
@xiaoyongzhu xiaoyongzhu deleted the xiaoyzhu/release_update_and_registry branch August 10, 2022 06:51
ahlag pushed a commit to ahlag/feathr that referenced this pull request Aug 26, 2022
…hr-ai#534)

* Update docs for UI and registry

* Update architecture.png

* Update feature-registry.md

* Update feathr-ui.png

* update docs

* Update feature-registry.md

* Update feature-registry.md

* update docs

* Update azure-deployment-arm.md

* Update azure-deployment-arm.md

* update

* update

* update pics

* Update feature-registry.md

* update on feathr UI

* fix broken links

* Update feature-registry.md

* resolve comments

* Update architecture.png

* Update feature-registry.md
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.

3 participants