Skip to content
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

Issue 2615 #2652

Closed
wants to merge 3 commits into from
Closed

Issue 2615 #2652

wants to merge 3 commits into from

Conversation

ddhuy
Copy link

@ddhuy ddhuy commented Nov 21, 2017

Solution: Add application filters in DAO layers

@ddhuy ddhuy closed this Nov 30, 2017
@gsaslis
Copy link
Contributor

gsaslis commented Nov 30, 2017

@ddhuy sorry for not getting back to you sooner on this.

Did you simply close this cause you're planning to open a new PR on the same topic?

@ddhuy
Copy link
Author

ddhuy commented Nov 30, 2017

@gsaslis . yes, i closed this request because it is so far out of dated with the master branch and has merging conflict. I am opening a new same & clean pull request now. Really sorry if I made you confused.

@ddhuy
Copy link
Author

ddhuy commented Nov 30, 2017

@gsaslis , new pull request is here #2678

@ddhuy ddhuy deleted the issue2615 branch November 30, 2017 17:19
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.

2 participants