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

Migrate GiftedChat to TypeScript #1190

Merged
merged 19 commits into from
Apr 30, 2019
Merged

Migrate GiftedChat to TypeScript #1190

merged 19 commits into from
Apr 30, 2019

Conversation

xcarpentier
Copy link
Collaborator

@xcarpentier xcarpentier commented Apr 5, 2019

Migrate the project to TypeScript.

cc @slorber @EvanBacon ;)

@xcarpentier xcarpentier self-assigned this Apr 5, 2019
@codecov
Copy link

codecov bot commented Apr 9, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@66fd0f8). Click here to learn what that means.
The diff coverage is 33.64%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1190   +/-   ##
=========================================
  Coverage          ?   34.99%           
=========================================
  Files             ?       20           
  Lines             ?      543           
  Branches          ?      181           
=========================================
  Hits              ?      190           
  Misses            ?      344           
  Partials          ?        9
Impacted Files Coverage Δ
src/Constant.ts 100% <ø> (ø)
src/MessageVideo.tsx 0% <0%> (ø)
src/__tests__/context.tsx 100% <100%> (ø)
src/GiftedChat.tsx 12.65% <12.65%> (ø)
src/GiftedAvatar.tsx 13.51% <13.51%> (ø)
src/MessageContainer.tsx 18.46% <18.46%> (ø)
src/Composer.tsx 20% <20%> (ø)
src/MessageText.tsx 33.33% <33.33%> (ø)
src/Actions.tsx 38.46% <38.46%> (ø)
src/Bubble.tsx 47.36% <47.36%> (ø)
... and 10 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 66fd0f8...85be40a. Read the comment docs.

@EvanBacon
Copy link
Contributor

thanks for this

@xcarpentier
Copy link
Collaborator Author

Hi @EvanBacon! What do you think if we change a bit the examples files structure and make a single app example by merging others?

And also do something like @slorber did every time when he makes an RN lib.
I mean an expo app at the root and an npm script before publish. It's really more easy for dev cycle of a lib! see it here:

@EvanBacon
Copy link
Contributor

@xcarpentier yeah, I think one example would be best. It'd also be great to make things like react-native-video optional now that Expo (specifically expo-av) has web support.

@xcarpentier
Copy link
Collaborator Author

OK, will do it in another PR.

@EvanBacon can you precise your suggestion about web support (ie. expo-av)?

@xcarpentier xcarpentier merged commit f247629 into master Apr 30, 2019
<View style={this.props.alignTop ? styles.containerAlignTop : styles.container}>
{this.state.showScrollBottom && this.props.scrollToBottom ? this.renderScrollToBottomWrapper() : null}
<FlatList
ref={(ref) => (this.flatListRef = ref)}
Copy link

Choose a reason for hiding this comment

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

It was good!

? this.renderScrollToBottomWrapper()
: null}
<FlatList
ref={this.flatListRef}
Copy link

Choose a reason for hiding this comment

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

Here is an error. This means that when sending a new message, do not scroll down. I am right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I you have time make PR, it will be more explicit. Thanks

@xcarpentier xcarpentier deleted the feature/typescript branch May 7, 2020 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants