Skip to content
This repository has been archived by the owner on Nov 28, 2022. It is now read-only.

feat: support multiple response types #208

Merged

Conversation

0xVolodya
Copy link
Contributor

@0xVolodya 0xVolodya commented Apr 14, 2019

closes #198
@domharrington this is ready. Can you check it when you available
https://github.com/readmeio/api-explorer/pull/208/files#diff-e7f8868abb312f633724097c3a090b40R61
I fixed a broken/wrong test here which came from the master

@0xVolodya
Copy link
Contributor Author

Peek 2019-04-14 22-01

Copy link
Member

@erunion erunion left a comment

Choose a reason for hiding this comment

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

Apologies for the delay on this. Just one small thing but otherwise this looks great!

return (
<div>
<select
style={{ backgroundColor: '#3d434a' }}
Copy link
Member

Choose a reason for hiding this comment

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

It would probably be best if this were a class name instead. Can you add a class for this into example/bundle-hub2.css? I'll make sure we do the same to our production CSS.

Copy link
Contributor Author

@0xVolodya 0xVolodya Jul 4, 2019

Choose a reason for hiding this comment

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

Thanks, I've added this

@0xVolodya 0xVolodya force-pushed the feat/suppoprt-multiple-response-types branch from 2b36ef6 to 87fbe42 Compare July 4, 2019 21:39
@0xVolodya 0xVolodya force-pushed the feat/suppoprt-multiple-response-types branch from 87fbe42 to c2f9f48 Compare July 4, 2019 21:47
@erunion erunion added the type:enhancement A potential new feature to be added, or an improvement we could make label Jul 8, 2019
Copy link
Member

@kanadgupta kanadgupta left a comment

Choose a reason for hiding this comment

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

@erunion looks good to me once we update that class name!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type:enhancement A potential new feature to be added, or an improvement we could make
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support multiple response types
4 participants