-
Notifications
You must be signed in to change notification settings - Fork 458
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
UBERF-4725 Migrate collaborative content #5717
base: main
Are you sure you want to change the base?
Conversation
// we don't want to show collaborative documents in activity | ||
// instead we show their content as Markup | ||
// TODO this should be generalized via activity extension | ||
if (attrClass === core.class.TypeCollaborativeDoc) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are you planning to do it in this pull request or is it a note for the future?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's decide on review how we handle this
I can do in this PR but It would be also good to have async trigger because file loading may affect the trigger performance.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can make activity generation as asynс trigger, I thought about that too
I would prefer to do it async in a separate pull request, and merge this pull request for now if it works
You can assign this task to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I figured out that I need to do a bit more changes to support mixins here. Will be done tomorrow.
7815ffd
to
728239e
Compare
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
…ments Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
Signed-off-by: Alexander Onnikov <[email protected]>
800165a
to
e683c9f
Compare
Huly®: UBERF-7136