-
Notifications
You must be signed in to change notification settings - Fork 113
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
feat: add ability to post comment to project messages #1322
Conversation
Images automagically compressed by Calibre's image-actions ✨ Compression reduced images by 21.4%, saving 936 bytes.
64 images did not require optimisation. |
…q/officelife into 2021-09-07-comments-to-tasks
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.
Great job, as always.
Not much to mention, except for the v-html
directive which here is dangerous!
app/Http/Controllers/Company/Company/Project/ProjectMessagesController.php
Outdated
Show resolved
Hide resolved
app/Http/Controllers/Company/Company/Project/ProjectMessagesController.php
Outdated
Show resolved
Hide resolved
Kudos, SonarCloud Quality Gate passed! |
🎉 This PR is included in version 0.5.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
This pull request has been automatically locked since there |
This PR adds the ability to add comments to... anything. We just started with project messages.
We've added a Comment object that can be morphed with
commentable
. We just need to add this association to other objects now to add comments to anything.Comments are a dedicated shared Vue class, so we can apply to anything we want.
The class takes a prop called
postUrl
. It's not super elegant, but it allows us to pass an endpoint and the class knows how to submit, edit and delete a comment based on this URL.