-
Notifications
You must be signed in to change notification settings - Fork 42
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
Service record configuration #52
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. wasn't able to run the complete flow against dev-env but I mocked a few responses to simulate some of the states to check and they all worked!
variation?: Variation | ||
}) => { | ||
let className = 'shadow rounded-sm p-2' | ||
className += ' shadow rounded-sm p-2' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't think this needs to be changed but FYI we also have a handy classNames
util in @/lib/utils
if you wanna avoid string concatenation like that.
* Ozone frontend/backend build and account setup (#35) * setup dockerfile and service entrypoint for frontend/backend deployed together * gitignore/yarnlock tidy * auth state for ozone configuration * update api package * plc/identity setup workflow * tidy/fixes * record creation flow on login * reorg client config / configuration flow * skip record creation step if not service account * tidy * rename metadata endpoint * fix * reorg plc constants, bring plc functionality into lib/identity * make generic ErrorInfo component * misc fixes * fix cypress * websocket fix * add docker workflow * update ozone version * fix entrypoint order of operations * fix next build / rel paths --------- Co-authored-by: Foysal Ahamed <[email protected]> * Docs on hosting ozone UI and backend (#48) * setup dockerfile and service entrypoint for frontend/backend deployed together * gitignore/yarnlock tidy * auth state for ozone configuration * update api package * plc/identity setup workflow * tidy/fixes * record creation flow on login * reorg client config / configuration flow * skip record creation step if not service account * tidy * rename metadata endpoint * fix * reorg plc constants, bring plc functionality into lib/identity * make generic ErrorInfo component * misc fixes * fix cypress * websocket fix * docs on hosting ozone ui and backend * minor tweaks * minor tweaks * add docker workflow * update ozone version * misc doc fixes * fix entrypoint order of operations * fix next build / rel paths * fix old ozone-ui references * fix markdown table * use a working build sha in compose * fix headers --------- Co-authored-by: Jake Gold <[email protected]> Co-authored-by: Foysal Ahamed <[email protected]> * Show custom labels from labeler service in ozone (#51) * setup dockerfile and service entrypoint for frontend/backend deployed together * gitignore/yarnlock tidy * auth state for ozone configuration * update api package * plc/identity setup workflow * tidy/fixes * record creation flow on login * reorg client config / configuration flow * skip record creation step if not service account * tidy * rename metadata endpoint * fix * reorg plc constants, bring plc functionality into lib/identity * make generic ErrorInfo component * misc fixes * fix cypress * websocket fix * add docker workflow * update ozone version * ✨ Use custom labeler labels in label input --------- Co-authored-by: Devin Ivy <[email protected]> * Service record configuration (#52) * setup dockerfile and service entrypoint for frontend/backend deployed together * gitignore/yarnlock tidy * auth state for ozone configuration * update api package * plc/identity setup workflow * tidy/fixes * record creation flow on login * reorg client config / configuration flow * skip record creation step if not service account * tidy * rename metadata endpoint * fix * reorg plc constants, bring plc functionality into lib/identity * make generic ErrorInfo component * misc fixes * fix cypress * websocket fix * boilerplate for service account configuration page * in-progess, configuration page * in-progess, configuration page * minor reorg in configure page * add react-json-view dep for record config * record configuration flow * fix dep * tidy * browser-only react viewer * build 0.1.0 rc * final tidy * ozone version bump * build * tidy --------- Co-authored-by: Foysal Ahamed <[email protected]> Co-authored-by: Jake Gold <[email protected]>
This adds a basic, low-level UI for modifying your labeler's service record using a structured JSON editor.