-
Notifications
You must be signed in to change notification settings - Fork 2
Make the project a react library depending on trame-iframe js lib #1
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
Conversation
jourdain
left a comment
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.
LGTM
2ff2c4f to
d8d0474
Compare
add a react patternfly app on how to use it license Apache 2
d8d0474 to
d571cb7
Compare
|
Can we at least merge this one even if we don't release anything? |
|
Are you asking me or @finetjul ? |
|
Sorry, asking both of you |
|
I'm fine merging that right away. |
finetjul
left a comment
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.
LGTM
see Add a JS lib to support trame-client API through an iframe trame-iframe#2