Skip to content

More features added: star, mark as read, bug fixes (including PR#131) #135

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

Closed
wants to merge 29 commits into from

Conversation

damms005
Copy link

The PR title says it all. More details in the commit change log :)

@simonj
Copy link

simonj commented Oct 15, 2018

Have you runned something like reformat code? seems like all the code has been touched but not all has changed

@cfpinto
Copy link
Collaborator

cfpinto commented Oct 31, 2018

@damms005 can you perform the changes requested, please? We would like to merge the suggested changes

@damms005
Copy link
Author

damms005 commented Nov 10, 2018

@cfpinto I'd like to re-PR these changes. Kindly address the issues in the comments

@damms005 damms005 force-pushed the feature_tag_conversation branch from 1ce63f5 to 51731fd Compare March 5, 2019 12:10
@damms005 damms005 changed the title More features added: star, mark as read, bug fixes (inluing to PR#131) More features added: star, mark as read, bug fixes (including PR#131) Mar 5, 2019
@damms005
Copy link
Author

damms005 commented Mar 5, 2019

@cfpinto @nahid All changes now effected according to the comments you made. Thanks

@@ -18,6 +21,39 @@
*/
class Adapter extends Service
{
<<<<<<< HEAD

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove Here

@damms005 damms005 closed this Jun 26, 2019
@damms005
Copy link
Author

damms005 commented Mar 28, 2020

This was closed in favor of PR #145

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.

5 participants