general clean up

Author: DanC5

README.md

@@ -164,8 +164,6 @@ Common mistakes that hurt performance are:
 - Mistake: Passing an object {} or an array [] that's not using seamless-immutable
 - Solution: Use an immutable type (i.e., a number or a string) or use a seamless immutable version of an object or an array
 
-## Bundle Size
-
 ## Internationalisation
 
 Instance of class `Streami18n` should be provided to the Chat component to handle translations.

emojiMartDataParser.js

@@ -11,14 +11,19 @@ const defaultDataSetJson = fs.readFileSync(
   path.join(__dirname, 'node_modules', 'emoji-mart', 'data', 'facebook.json'),
   'utf8',
 );
+
 const dataSet = JSON.parse(defaultDataSetJson);
+
 const removeSkinTonesInfo = (emojiKey) => {
   if (dataSet.emojis[emojiKey].skin_variations) {
     delete dataSet.emojis[emojiKey].skin_variations;
   }
 };
+
 Object.keys(dataSet.emojis).map(removeSkinTonesInfo);
+
 const newDataSetJson = JSON.stringify(dataSet);
+
 fs.writeFileSync(
   path.join(__dirname, 'src', 'stream-emoji.json'),
   newDataSetJson,

src/components/Chat/__tests__/Chat.test.js

@@ -22,6 +22,7 @@ const TranslationContextConsumer = ({ fn }) => {
   fn(useContext(TranslationContext));
   return <div data-testid="children" />;
 };
+
 describe('Chat', () => {
   afterEach(cleanup);
   const chatClient = getTestClient();
@@ -182,9 +183,7 @@ describe('Chat', () => {
 
   describe('mutes', () => {
     it('init the mute state with client data', async () => {
-      const chatClientWithUser = await getTestClientWithUser({
-        id: 'user_x',
-      });
+      const chatClientWithUser = await getTestClientWithUser({ id: 'user_x' });
       // First load, mutes are initialized empty
       chatClientWithUser.user.mutes = [];
       let context;
@@ -215,9 +214,7 @@ describe('Chat', () => {
     });
 
     it('chat client listens and updates the state on mute event', async () => {
-      const chatClientWithUser = await getTestClientWithUser({
-        id: 'user_x',
-      });
+      const chatClientWithUser = await getTestClientWithUser({ id: 'user_x' });
 
       let context;
       render(

src/mock-builders/emojiContext.js renamed to src/mock-builders/generator/emoji.js

@@ -5,8 +5,8 @@ import {
   commonEmoji,
   defaultMinimalEmojis,
   emojiSetDef,
-} from '../components/Channel/emojiData';
-import emojiData from '../stream-emoji.json';
+} from '../../components/Channel/emojiData';
+import emojiData from '../../stream-emoji.json';
 
 export const emojiMockConfig = {
   emojiData,

src/mock-builders/generator/index.js

@@ -1,6 +1,7 @@
+export * from './attachment';
 export * from './channel';
-export * from './message';
-export * from './user';
+export * from './emoji';
 export * from './member';
+export * from './message';
 export * from './reaction';
-export * from './attachment';
+export * from './user';

src/mock-builders/index.js

@@ -50,4 +50,3 @@ export const getRandomInt = (min, max) => {
 export * from './api';
 export * from './event';
 export * from './generator';
-export * from './emojiContext';

types/index.d.ts

@@ -91,9 +91,9 @@ export interface EmojiConfig {
   emojiData: EmojiMartData;
   commonEmoji: commonEmojiInterface;
   defaultMinimalEmojis: MinimalEmojiInterface[];
-  EmojiPicker: React.ElementType<NimblePickerProps> | null;
-  EmojiIndex: NimbleEmojiIndex | null;
-  Emoji: React.ElementType<NimbleEmojiProps> | null;
+  EmojiPicker: React.ElementType<NimblePickerProps>;
+  EmojiIndex: NimbleEmojiIndex;
+  Emoji: React.ElementType<NimbleEmojiProps>;
   emojiSetDef: {
     spriteUrl: string;
     size: number;
@@ -189,10 +189,10 @@ export interface ChatProps {
 
 export interface ChannelProps {
   channel?: Client.Channel;
-  Emoji: React.ElementType<NimbleEmojiProps> | null;
-  emojiData: EmojiMartData;
-  EmojiIndex: NimbleEmojiIndex | null;
-  EmojiPicker: React.ElementType<NimblePickerProps> | null;
+  Emoji?: React.ElementType<NimbleEmojiProps>;
+  emojiData?: EmojiMartData;
+  EmojiIndex?: NimbleEmojiIndex;
+  EmojiPicker?: React.ElementType<NimblePickerProps>;
   /** The loading indicator to use */
   LoadingIndicator?: React.ElementType<LoadingIndicatorProps>;
   LoadingErrorIndicator?: React.ElementType<LoadingErrorIndicatorProps>;