Skip to content

Meteor update#1338

Merged
brylie merged 41 commits intodevelopfrom
meteor-update
Aug 10, 2016
Merged

Meteor update#1338
brylie merged 41 commits intodevelopfrom
meteor-update

Conversation

@55
Copy link
Copy Markdown
Contributor

@55 55 commented Jul 21, 2016

Closes #1318
Closes #1351

Updated to v1.4.0.1
Please test and merge.

Release notes - https://github.com/meteor/meteor/blob/devel/History.md

@55 55 added the in progress label Jul 21, 2016
@55 55 added this to the Sprint 27 milestone Jul 21, 2016
@55 55 added the WIP label Jul 21, 2016
@55
Copy link
Copy Markdown
Contributor Author

55 commented Jul 21, 2016

WIP. getting this error:

While minifying app stylesheet:
packages/minifier-css/minifier.js:184:1: Cannot call method 'replace' of null
  at toOSPath (packages/minifier-css/minifier.js:184:1)
  at pathRelative (packages/minifier-css/minifier.js:205:1)
  at packages/minifier-css/minifier.js:164:1
  at Array.forEach (native)
  at Function._.each._.forEach (packages/underscore/underscore.js:105:1)
  at packages/minifier-css/minifier.js:122:1
  at Array.forEach (native)
  at Function._.each._.forEach (packages/underscore/underscore.js:105:1)
  at rewriteRules (packages/minifier-css/minifier.js:99:1)
  at Object.CssTools.rewriteCssUrls (packages/minifier-css/minifier.js:85:1)
  at packages/minifier-css/minifier.js:55:1
  at Array.forEach (native)
  at Function._.each._.forEach (packages/underscore/underscore.js:105:1)
  at Object.CssTools.mergeCssAsts (packages/minifier-css/minifier.js:32:1)
  at packages/minifyStdCSS/plugin/minify-css.js:79:1
  at CssToolsMinifier.processFilesForBundle (packages/minifyStdCSS/plugin/minify-css.js:17:1)

Created an issue - meteor/meteor#7446

@philippeluickx
Copy link
Copy Markdown
Contributor

Getting the next issue:

[[[[[ ~/Development/projects/sampo/apinf/api-umbrella-dashboard ]]]]]

=> Started proxy.
=> Started MongoDB.
I20160729-12:25:23.939(3)? { [Error: Cannot find module '../build/Release/bson'] code: 'MODULE_NOT_FOUND' }
W20160729-12:25:24.189(3)? (STDERR) js-bson: Failed to load c++ bson extension, using pure JS version
I20160729-12:25:27.940(3) (synced-cron-server.js:63) SyncedCron: Scheduled "Sync API Umbrella Users and API Backends" next run @FRI Jul 29 2016 12:25:27 GMT+0300 (EEST)
=> Started your app.

=> App running at: http://localhost:3000/

@philippeluickx
Copy link
Copy Markdown
Contributor

Issue is linked to deprecated cfs package, see discussion: https://forums.meteor.com/t/deprecating-collectionfs/16921/2

Migrating to https://github.com/vsivsi/meteor-file-collection to avoid future problems.

@55
Copy link
Copy Markdown
Contributor Author

55 commented Aug 1, 2016

@philippeluickx, removing cfs and everything related to it didn't help.

@philippeluickx
Copy link
Copy Markdown
Contributor

@NNN Some other package then? Try to remove all packages and add them back in one by one...?

@55
Copy link
Copy Markdown
Contributor Author

55 commented Aug 1, 2016

@philippeluickx, it would be great if you can help with it 😄

@philippeluickx
Copy link
Copy Markdown
Contributor

Working on it

@55 55 assigned philippeluickx and unassigned frenchbread Aug 2, 2016
@philippeluickx
Copy link
Copy Markdown
Contributor

Linked to old cfs packages.
See Meteor-Community-Packages/Meteor-CollectionFS#984

I propose that we get rid of all the "old" file-related packages and refactor code to use 1, well-maintained library - https://github.com/vsivsi/meteor-file-collection
Open backlog issue: https://github.com/apinf/api-umbrella-dashboard/issues/1351

I branched out with a removed cfs and comment out code where this was still used. Everything runs now on my machine - https://github.com/apinf/api-umbrella-dashboard/tree/meteor-update-removed-cfs
I did follow https://guide.meteor.com/1.4-migration.html#binary-packages-require-build-toolchain and ran
meteor npm rebuild
So this might also be necessary

@55 55 self-assigned this Aug 3, 2016
@brylie brylie modified the milestones: Sprint 28, Sprint 27 Aug 8, 2016
@brylie brylie self-assigned this Aug 8, 2016
@55 55 removed the WIP label Aug 9, 2016
@55
Copy link
Copy Markdown
Contributor Author

55 commented Aug 9, 2016

@brylie, please review.

@brylie
Copy link
Copy Markdown
Contributor

brylie commented Aug 10, 2016

I just ran meteor update once more. There were about 5 additional updates.

@brylie
Copy link
Copy Markdown
Contributor

brylie commented Aug 10, 2016

When visiting the landing page, I get the following error:

Exception in template helper: TypeError: Cannot read property 'projectLogoFileId' of undefined
    at Object.projectLogoExists

@brylie
Copy link
Copy Markdown
Contributor

brylie commented Aug 10, 2016

We may need a try/catch around the code that uses projectLogoFileId, or to pre-verify that the branding object is not undefined before using one of its properties.

@brylie
Copy link
Copy Markdown
Contributor

brylie commented Aug 10, 2016

It seems like there is a bug in the projectLogoExists template helper. It is throwing the above error even when I have uploaded a project logo.

@brylie brylie merged commit 604cab8 into develop Aug 10, 2016
@brylie brylie deleted the meteor-update branch August 10, 2016 13:09
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.

4 participants