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

AddToCalendar migration #4

Merged
merged 14 commits into from
Nov 19, 2020
Merged

AddToCalendar migration #4

merged 14 commits into from
Nov 19, 2020

Conversation

kuus
Copy link
Contributor

@kuus kuus commented Nov 17, 2020

Migrated AddToCalendar component to typescript, for this I had to add the resolutions key in the package.json as suggested in this issue in order to prevent tsdx lint to break on commit prehook.

small notes:

  • removed the external react-add-to-calendar dep and add just two small deps:
    • is-mobile (as I don't think we want to mantain a list of browser vendors :D)
    • the tiny uuid (with mock for jest)
  • replaced the formatting previously done with momentjs within react-add-to-calendar with date-fns and fixed outlookcom url building.
  • everything is typed here now

* 'master' of github.com:GetStream/mml-react:
  fix rollup named exports
* 'master' of github.com:GetStream/mml-react: (46 commits)
  first story
  remove styleguidist
  rename converterConfig
  remove old docs folder
  convert parser to typescript
  converterConfig to typescript
  convert test files to tsx
  remove ts ignores
  refactor tags
  simplify parser
  Scheduler support selected attr
  snapshot examples
  fix converterConfig recursive call
  enable ci test
  fix babel test config
  jest config
  remove style import from carousel
  comment out LoadingIndicator in Loader component
  convert mml test to parser test
  parser to ts
  ...
… dep, fix outlookcom url construction, get rid of yahoo followinf source code comment, I had to add the resolutions key to prevent tsdx linting from erroring
@github-actions
Copy link

github-actions bot commented Nov 17, 2020

Size Change: +983 B (1%)

Total Size: 57.3 kB

Filename Size Change
./dist/mml-react.cjs.development.js 21.2 kB +354 B (1%)
./dist/mml-react.cjs.production.min.js 10.6 kB +283 B (2%)
./dist/mml-react.esm.js 21.1 kB +346 B (1%)
ℹ️ View Unchanged
Filename Size Change
./dist/index.css 4.19 kB 0 B
./dist/index.js 146 B 0 B

compressed-size-action

Copy link
Contributor

@mahboubii mahboubii left a comment

Choose a reason for hiding this comment

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

I pushed some stuff to simplify things a bit, other than that it looks good

jest-global-setup.js Outdated Show resolved Hide resolved
src/components/AddToCalendar.tsx Show resolved Hide resolved
LOCATION:
END:VEVENT
END:VCALENDAR"
href="https://outlook.live.com/owa/?rru=addevent&startdt=2019-12-24T14%3A42%3A54Z&enddt=2019-12-24T15%3A42%3A54Z&subject=Massage%20with%20Jessica&allday=false&uid=00000000-0000-0000-0000-000000000000&path=/calendar/view/Month"
Copy link
Contributor

Choose a reason for hiding this comment

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

@kuus is this a valid link here?

LOCATION:
END:VEVENT
END:VCALENDAR"
href="https://calendar.google.com/calendar/render?action=TEMPLATE&dates=20191224T144254Z/20191224T154254Z&text=Massage%20with%20Jessica"
Copy link
Contributor

Choose a reason for hiding this comment

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

same here?

@mahboubii mahboubii merged commit 85dc43d into master Nov 19, 2020
@delete-merged-branch delete-merged-branch bot deleted the addtocalendar branch November 19, 2020 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants