+ + Editor embeds merge in nitro embed + + #960 +
+Conversation
+
+
+
+ appinteractive
+ and others
+ added some commits
+
+ on Dec 27, 2018
+
+
+
+
+
+ mattwr18
+ and others
+ added some commits
+
+ 19 days ago
+
+
+
+
+
+ This comment has been minimized.
+
+
+ Show comment
+ Hide comment
+
+
+ + This comment has been minimized. + +
+
+
+
+
+ codecov
+ bot
+
+
+
+ 18 days ago
+
+
+
+
+
+ Codecov Report
++++
No coverage uploaded for pull request base ( master@324330a). Click here to learn what that means.
+The diff coverage isn/a.
+
+
+
+
+
+ codecov
+ bot
+
+
+
+ commented
+
+
+ 18 days ago
+
+
+
+
+
+Choose a reason for hiding this comment
+The reason will be displayed to describe this comment to others. Learn more.
+
+ Codecov Report+++ |
+
+
+
+ mattwr18
+
+
+ referenced this pull request
+
+ 16 days ago
+
+
+
+
+
+
+ Closed
+
+
+
+
+
+
+ + + Editor embeds + #953 +
+ + + + + + + +
+
+
+ This comment has been minimized.
+
+
+ Show comment
+ Hide comment
+
+
+ + This comment has been minimized. + +
+
+
+
+
+ mattwr18
+
+
+
+
+ 9 days ago
+
+
+
+
+ Member
+
+
+
+ Author
+
+
+ does this look ok @roschaefer? I used tomono, but because I already had a repo with the name Human-Connection, I needed to remove it and recreate it with the merged commit history. This caused some small issues for me locally, but I think others won't face the same.
The files changed and commit history look good to me
+
+
+
+
+
+
+ mattwr18
+
+
+
+
+ commented
+
+
+ 9 days ago
+
+
+
+
+
+Choose a reason for hiding this comment
+The reason will be displayed to describe this comment to others. Learn more.
+|
+ does this look ok @roschaefer? I used The files changed and commit history look good to me + |
+
+
+
+ This comment has been minimized.
+
+
+ Show comment
+ Hide comment
+
+
+ + This comment has been minimized. + +
+
+
+
+
+ roschaefer
+
+
+
+
+ 5 days ago
+
+
+
+
+ Member
+
+
+
+ @mattwr18 this looks good, but could sb. explain me why this is a separate service and not just another resolver in our backend?
+@appinteractive maybe?
+
+
+
+
+
+
+ roschaefer
+
+
+
+
+ commented
+
+
+ 5 days ago
+
+
+
+
+
+Choose a reason for hiding this comment
+The reason will be displayed to describe this comment to others. Learn more.
+|
+ @mattwr18 this looks good, but could sb. explain me why this is a separate service and not just another resolver in our backend? +@appinteractive maybe? + |
+
+
+
+ This comment has been minimized.
+
+
+ Show comment
+ Hide comment
+
+
+ + This comment has been minimized. + +
+
+
+
+
+ mattwr18
+
+
+
+
+ 5 days ago
+
+
+
+
+ Member
+
+
+
+ Author
+
+
+ I think it could be, I thought the same thing myself @roschaefer
+
+
+
+
+
+
+ mattwr18
+
+
+
+
+ commented
+
+
+ 5 days ago
+
+
+
+
+
+Choose a reason for hiding this comment
+The reason will be displayed to describe this comment to others. Learn more.
+|
+ I think it could be, I thought the same thing myself @roschaefer + |
+
+
+
+ roschaefer
+
+ added some commits
+
+ 2 days ago
+
+
+ Merge state
+Add more commits by pushing to the 256-editor-embeds-merge-in-nitro-embed branch on Human-Connection/Human-Connection.
+ Review required +
+ + + + +All checks have passed
+ +Couldn’t update branch
+Oops, something went wrong.
+ +
+ + +
+ ++ +
++ + + + + + mattwr18 + + + + + commented + + +18 days ago
+
+
+
+
+
+
Pullrequest
+ +Issues
+ ++- fixes #256
+
+