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

add cloud version #5198

Merged
merged 23 commits into from Aug 9, 2021
Merged

add cloud version #5198

merged 23 commits into from Aug 9, 2021

Conversation

jamakase
Copy link
Contributor

@jamakase jamakase commented Aug 4, 2021

What

Adds some new concepts into oss version:

  • middleware for all services and resources
  • feature service

Adds cloud version

@cgardens
Copy link
Contributor

cgardens commented Aug 6, 2021

@jamakase when can we merge this. I know not everything is done but maybe we can do other features and bug fixes in a separate PR?

const Title = styled(H5)`
padding: 25px 25px 22px;
border-bottom: 1px solid ${({ theme }) => theme.greyColor20};
//border-bottom: 1px solid ${({ theme }) => theme.greyColor20};
Copy link
Contributor

Choose a reason for hiding this comment

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

@jamakase should this be removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually no, this is part of new UI where we update borders and shadows. Decided to leave it as a comment to check all places more properly.

Copy link
Contributor

@davinchia davinchia left a comment

Choose a reason for hiding this comment

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

I think I spotted one typo, otherwise I did a very cursory glance through.

@jamakase jamakase merged commit 219877f into master Aug 9, 2021
@jamakase jamakase deleted the jamakase/add-cloud-version branch August 9, 2021 13:04
@jamakase jamakase changed the title Jamakase/add cloud version add cloud version Aug 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants