8000 Exception while invoking method 'inviteUserToBoard' Error: Is comment only is required · Issue #950 · wekan/wekan · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Exception while invoking method 'inviteUserToBoard' Error: Is comment only is required #950

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.

Sign up for GitHub

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

Jump to bottom
Closed
vampirefrog opened this issue Mar 29, 2017 · 8 comments
8000
Closed

Exception while invoking method 'inviteUserToBoard' Error: Is comment only is required #950

vampirefrog opened this issue Mar 29, 2017 · 8 comments

Comments

@vampirefrog
Copy link

in boards.js I've had to add 'isCommentOnly: true' around line 315, to get rid of this error. Not only that, but when inviting a user to a board, the only error I would receive was '400'
@rhelsing
Copy link
Contributor

@vampirefrog I implemented the comment permissions feature but I am fairly new to meteor. I did not experience this issue when testing but I will try to fix it if you give me more details.. If the default level that a user should be at is normal when a user is invited, then isCommentOnly should be set to false instead.

@rhelsing rhelsing mentioned this issue Mar 29, 2017
Merged
@vampirefrog
Copy link
Author

Sorry, I meant false indeed. Thanks for the quick fix

@xet7
Copy link
Member
xet7 commented Mar 30, 2017

Merged. Will be included in next release.

@xet7 xet7 closed this as completed Mar 30, 2017
@neooleg
Copy link
neooleg commented Apr 2, 2017 •
edited
Loading

Abstract

@rhelsing I have your hotfix (my revision is 823aea4), but the issue is still reproduces.

Do you need more details to investigate?

STR

  1. Try to import Trello using manual https://github.com/wekan/wekan/wiki/Migrating-from-Trello
  2. Map user as Wekan asked
  3. Go next

AR

There is span with "400" error inside.

Stack trace is the following:

I20170402-17:07:38.220(3)? Exception while invoking method 'importTrelloBoard' Error: Is comment only is required
I20170402-17:07:38.382(3)? at getErrorObject (packages/aldeed_collection2-core/lib/collection2.js:437:1)
I20170402-17:07:38.383(3)? at [object Object].doValidate (packages/aldeed_collection2-core/lib/collection2.js:420:1)
I20170402-17:07:38.383(3)? at [object Object]..each.Mongo.Collection.(anonymous function) [as insert] (packages/aldeed_collection2-core/lib/collection2.js:173:1)
I20170402-17:07:38.383(3)? at CollectionHooks.extendCollectionInstance..each.self.direct.(anonymous function) (packages/matb33_collection-hooks/collection-hooks.js:98:1)
I20170402-17:07:38.383(3)? at [object Object]..extend.withValue (packages/meteor/dynamics_nodejs.js:56:1)
I20170402-17:07:38.383(3)? at Object.directOp (packages/matb33_collection-hooks/collection-hooks.js:18:1)
I20170402-17:07:38.383(3)? at Object.CollectionHooks.extendCollectionInstance..each.self.direct.(anonymous function) [as insert] (packages/matb33_collection-hooks/collection-hooks.js:97:1)
I20170402-17:07:38.383(3)? at TrelloCreator.createBoardAndLabels (models/import.js:189:35)
I20170402-17:07:38.384(3)? at [object Object].importTrelloBoard (models/import.js:497:35)
I20170402-17:07:38.384(3)? at packages/check/match.js:107:1
I20170402-17:07:38.384(3)? at [object Object]..extend.withValue (packages/meteor/dynamics_nodejs.js:56:1)
I20170402-17:07:38.384(3)? at Object.meteorInstall.node_modules.meteor.check.match.js.exports.Match.failIfArgumentsAreNotAllChecked (packages/check/match.js:106:1)
I20170402-17:07:38.384(3)? at maybeAuditArgumentChecks (packages/ddp-server/livedata_server.js:1708:18)
I20170402-17:07:38.384(3)? at meteorInstall.node_modules.meteor.ddp-server.livedata_server.js..extend.protocol_handlers.method.payload.msg (packages/ddp-server/livedata_server.js:711:19)
I20170402-17:07:38.384(3)? at [object Object]..extend.withValue (packages/meteor/dynamics_nodejs.js:56:1)
I20170402-17:07:38.385(3)? at meteorInstall.node_modules.meteor.ddp-server.livedata_server.js._.extend.protocol_handlers.method.payload.msg (packages/ddp-server/livedata_server.js:709:40)
I20170402-17:07:38.387(3)? Sanitized and reported to the client as: Is comment only is required [400]
I20170402-17:07:38.387(3)?

AR

The board should be created and all data imported

@xet7 xet7 reopened this Apr 2, 2017
@rhelsing
Copy link
Contributor
rhelsing commented Apr 4, 2017 •
edited
Loading

@xet7 would you recommend that I do something like in #970 to resolve this? Except with isCommentOnly? aka set isCommentOnly to false on user create? do I need to create a migration too?

@neooleg neooleg mentioned this issue Apr 15, 2017
Closed
@rhelsing
Copy link
Contributor

@xet7 see above?

@Jean-Daniel
Copy link

Note that there is at least 2 places where user are created while importing Trello Boards.

The first one is at wekan/import.js:144

The second one at wekan/import.js:169

@rhelsing
Copy link
Contributor

@Jean-Daniel thank you, are there any other spots I may have missed? Is there a callback or modification I can make at a higher level (on the user model maybe?) that would solve all of these potential issues? Something similar to #970

@rhelsing rhelsing mentioned this issue Apr 18, 2017
Merged
@xet7 xet7 closed this as completed Apr 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@xet7 @vampirefrog @Jean-Daniel @rhelsing @neooleg
0