actually change the button href (#420)

src/components/Meeting.tsx

@@ -188,7 +188,7 @@ export default function Meeting({
       });
     if (meeting[`contact_${i}_phone` as keyof typeof Meeting])
       contactButtons.push({
-        href: `tel:${meeting[`contact_${i}_phone` as keyof typeof Meeting]}`,
+        href: `sms:${meeting[`contact_${i}_phone` as keyof typeof Meeting]}`,
         icon: 'text',
         text: i18n(strings.contact_text, {
           contact: meeting[`contact_${i}_name` as keyof typeof Meeting],

test/__tests__/Meeting.spec.tsx

@@ -47,6 +47,11 @@ describe('<Meeting />', () => {
     conference_phone_notes: 'Test',
     conference_provider: 'Zoom',
     updated: '2/17/22',
+    contact_1_name: 'Contact 1',
+    contact_1_email: '[email protected]',
+    contact_1_phone: '+18005551212',
+    contact_2_email: '[email protected]',
+    contact_2_phone: '+18005551212',
   };
 
   const mockState: State = {
@@ -140,6 +145,31 @@ describe('<Meeting />', () => {
     expect(container).toBeTruthy();
   });
 
+  it('renders with contact 1 but no contact 2', () => {
+    const { getByText, queryByText } = render(
+      <MemoryRouter>
+        <Meeting
+          state={mockState}
+          setState={jest.fn()}
+          mapbox="pk.123456"
+          feedback_emails={['[email protected]']}
+        />
+      </MemoryRouter>
+    );
+    const contact1text = getByText(`Text ${mockMeeting.contact_1_name}`);
+    expect(contact1text).toHaveAttribute(
+      'href',
+      `sms:${mockMeeting.contact_1_phone}`
+    );
+    const contact1email = getByText(`Email ${mockMeeting.contact_1_name}`);
+    expect(contact1email).toHaveAttribute(
+      'href',
+      `mailto:${mockMeeting.contact_1_email}`
+    );
+    const contact2text = queryByText(`Text ${mockMeeting.contact_2_name}`);
+    expect(contact2text).toBeNull();
+  });
+
   it('renders when inactive', () => {
     const { container } = render(
       <MemoryRouter>