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

Revised Goals #70

Merged
merged 1 commit into from
Feb 6, 2015
Merged

Revised Goals #70

merged 1 commit into from
Feb 6, 2015

Conversation

dustinfarris
Copy link
Owner

Going to leave this open to comment for a few days.

/cc @benkonrath @holandes22

@benkonrath
Copy link
Collaborator

I originally wrote this to explain why I wanted to start from scratch instead of trying to continue with the old adapter. My personal goal for EDA is to ensure we continue to follow the officially supported way to create custom adapter in ED. I think the updated goals covers this and also solidifies the fact that we're interested in helping to fix issues in ED and in DRF if required.

I'm fine with this change. Thanks for taking the time to update this.

@holandes22
Copy link
Collaborator

Nice goals indeed.

I really apologize guys I wasn't around to help, I was determined to help with this since I use it for personal projects, but my bandwidth got very limited lately due to work (unfortunately we don't use ember at work so wasn't able to squeeze work on EDA at my day job). I should be having more free time in 3 weeks, so I plan to help with any open issues to get ready for ED 1.0 release.

Being of course following, and just want to say that you did an amazing job on the implementation, tests and docs, kudos guys!

@dustinfarris
Copy link
Owner Author

Thanks, @holandes22! No worries on the limited bandwidth—we can definitely sympathize with that!

@benkonrath
Copy link
Collaborator

@holandes22 Yeah, no problem on being busy - we appreciated your help in the beginning. :)

@dustinfarris Is this good to merge?

@dustinfarris
Copy link
Owner Author

@benkonrath yes, I think we're all in agreement.

benkonrath added a commit that referenced this pull request Feb 6, 2015
@benkonrath benkonrath merged commit 6959c5f into master Feb 6, 2015
@dustinfarris dustinfarris deleted the goals branch February 6, 2015 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants