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

Release v2.16.0 #1085

Merged
merged 1 commit into from
May 8, 2019
Merged

Release v2.16.0 #1085

merged 1 commit into from
May 8, 2019

Conversation

Gnito
Copy link
Contributor

@Gnito Gnito commented May 8, 2019

This release makes 2 big updates to sharetribe-scripts package (which is our fork from Create
React App). It is updated from v1.1.5 -> 2.1.8 -> 3.0.0. This brought up
a couple of changes:

  • package.json has now a "browserlist" configuration key. This gives you an option to affect
    browser support (it affects CSS Autoprefixer and JS build output).
    You might want to update it.
  • IE support is removed from Create React App, but you can add polyfills yourself if needed.
  • React was updated to a version that supports Hooks and Rules of React eslint plugin is
    included.
  • All the npm vulnerability report exceptions were removed from .auditrc

There was also a couple of bug fixes you should check carefully:
#1082, #1084.

Changes:

  • [fix] Previous change from currentUser.attributes.stripeConnected to separately included
    stripeAccount caused errors since updates to currentUser entity didn't include stripeAccount.
    Including it every time sounds quite error-prone, so we reversed that change.
    #1084
  • [fix] Edit updatedEntities function in util/data.js so that it doesn't mutate the
    oldEntities argument. #1079
  • [change] Update sharetribe-scripts (CRA fork) to v3.0.0. There are a couple of changes that you
    should check from #1081
    • Reserve use* function naming pattern for React Hooks.
    • Recent SDK update changed the proptypes for snapshots.
    • Updated scripts/config.js after Prettier version bump
    • Removed unnecessary audit exceptions
  • [fix] Ensure on TransactionPage that all the required data is loaded before showing the page.
    #1082
  • [fix] Use proper method for Sentry on logout to avoid error message.
    #1080,
  • [change] Update sharetribe-scripts (CRA fork) to v2.1.8. There are a couple of changes that you
    should check from #1073
    • package.json has now a "browserlist" configuration key. This gives you an option to affect
      Autoprefixer configs (aka CSS vendor prefixes / browser support). You might want to update it.
    • IE support is removed from Create React App, but you can add polyfills yourself if needed.
    • Test snapshots were also changed a bit. (Update your own custom tests if needed.)
    • Some CSS and React rules were more strict, you might need to make changes to your custom code.
  • [fix] New npm vulnerability alerts checked and added to exception list.
    #1075
  • [fix] ListingPage.duck: fix minor bug on dispatching the fetchReviewsRequest action
    #1074

@Gnito Gnito merged commit 47e6f10 into master May 8, 2019
@Gnito Gnito deleted the v2.16.0 branch May 9, 2019 14:16
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.

1 participant