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

Bump requirement for two packages #1

Merged
merged 1 commit into from
Feb 22, 2022
Merged

Conversation

xiaoyongzhumsft
Copy link
Contributor

No description provided.

@xiaoyongzhumsft xiaoyongzhumsft marked this pull request as draft February 19, 2022 07:36
@xiaoyongzhumsft xiaoyongzhumsft added the bug Something isn't working label Feb 22, 2022
@xiaoyongzhumsft xiaoyongzhumsft marked this pull request as ready for review February 22, 2022 11:19
@hangfei hangfei merged commit 1164711 into main Feb 22, 2022
@jaymo001 jaymo001 deleted the requirement_version_bump branch March 7, 2022 07:33
blrchen added a commit that referenced this pull request May 5, 2022
* Init react based feathr ui code (#1)

* Enables continues integration and deployment to cloud for ui app (#3)

This PR enables continues integration and deployment to cloud for ui app

- Manually create azure web app under ACE subscription resource group feathr-ui-test
- Create github action workflow to trigger deployment on main branch push or pull request
- Deploy policy
   - Main branch will be deployed to https://thankful-mud-0addb6c00.1.azurestaticapps.net/
   - Each pull request will be deployed to a staging environment, the url will be generated dynamically n each PR's comment.

* Add UI landing page

* Workaround for CI treat warning as error

* Enable client side route in Azure Static Web App

* Code clean up

* Code clean up

* Enable eslint rules

* Update README.md

* Fix CI error

* Add data source mockup page

* Enable Azure AD login

* Complete api integration

* Complete api integration

* Show lineage in feature detail page

* Resolve demo review comments

* Update page title

* Add lineage UI page

* Update Web UI CICD file name and secret name

* Workaround for PR from forked branch can not access main repo's secret

* Workaround for PR from forked branch can not access main repo's secret

* Resolve code review comments

* Test use main repo's secret

* Revert "Test use main repo's secret"

This reverts commit 0ca7f07.

* Remove two more hard codings
tianz1113 pushed a commit to tianz1113/feathr that referenced this pull request Sep 11, 2023
first attempt at adapting feathr to cccm use cases
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants