Skip to content

Feature/meteor update#1162

Merged
55 merged 30 commits intodevelopfrom
feature/meteor-update
Jun 29, 2016
Merged

Feature/meteor update#1162
55 merged 30 commits intodevelopfrom
feature/meteor-update

Conversation

@brylie
Copy link
Copy Markdown
Contributor

@brylie brylie commented Jun 15, 2016

Closes #1161

Changes

  • upgrade all project dependencies
  • upgrade to Meteor 1.3.3
  • move Clipboard dependency to npm

@brylie brylie added this to the Sprint 24 milestone Jun 15, 2016
@55 55 added the WIP label Jun 21, 2016
@55
Copy link
Copy Markdown
Contributor

55 commented Jun 21, 2016

While testing this branch, I have found:

/profile:

  • apiKey is missing

@brylie
Copy link
Copy Markdown
Contributor Author

brylie commented Jun 28, 2016

I have updated all packages again, including Meteor 1.3.4. @apinf/core please review.

@brylie brylie removed the WIP label Jun 28, 2016
@brylie
Copy link
Copy Markdown
Contributor Author

brylie commented Jun 28, 2016

The missing api key is related to an open bug report:
https://github.com/apinf/api-umbrella-dashboard/issues/1216

It was not introduced as part of this upgrade.

@55
Copy link
Copy Markdown
Contributor

55 commented Jun 28, 2016

@brylie, reviewing this.

@55
Copy link
Copy Markdown
Contributor

55 commented Jun 28, 2016

Getting on Windows 10:

=> Errors prevented startup:

   While building for web.browser:
   apis/collection/backend.js:1:83: Unexpected token (1:83) 

Will check with @philippeluickx if it just a Windows 10 thing.

@bajiat bajiat modified the milestones: Sprint 25, Sprint 24 Jun 28, 2016
@55
Copy link
Copy Markdown
Contributor

55 commented Jun 28, 2016

@philippeluickx, tested myself, everything works as expected. Test yourself and merge it.

@philippeluickx
Copy link
Copy Markdown
Contributor

Catalog is empty for me, @NNN looking into it.

@philippeluickx
Copy link
Copy Markdown
Contributor

Still empty...

@philippeluickx
Copy link
Copy Markdown
Contributor

I20160628-17:57:08.005(3) (synced-cron-server.js:63) SyncedCron: Exception "Sync API Umbrella Users and API Backends" ReferenceError: ApiBackends is not defined
at apis/server/methods/apiUmbrella.js:12:34
at Function..each..forEach (packages/underscore/underscore.js:108:1)
at [object Object].syncApiBackends (apis/server/methods/apiUmbrella.js:10:9)
at maybeAuditArgumentChecks (packages/ddp-server/livedata_server.js:1704:12)
at packages/ddp-server/livedata_server.js:1617:18
at [object Object]._.extend.withValue (packages/meteor/dynamics_nodejs.js:56:1)
at Server.apply (packages/ddp-server/livedata_server.js:1616:45)
at Server.call (packages/ddp-server/livedata_server.js:1559:17)
at Object.job (server/startup.js:18:16)
at packages/percolate_synced-cron/synced-cron-server.js:218:1

@55 55 merged commit 553a40c into develop Jun 29, 2016
@55 55 deleted the feature/meteor-update branch June 29, 2016 09:51
@55 55 removed the in progress label Jun 29, 2016
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