Skip to content

Add urls and views#20

Open
jjdelc wants to merge 8 commits into
hiidef:draft-16-masterfrom
jjdelc:add-urls-and-views
Open

Add urls and views#20
jjdelc wants to merge 8 commits into
hiidef:draft-16-masterfrom
jjdelc:add-urls-and-views

Conversation

@jjdelc
Copy link
Copy Markdown

@jjdelc jjdelc commented Mar 14, 2012

Adding oauth2app a views module and pluggable url patterns.

Updated the example project to use those.

@bhagany
Copy link
Copy Markdown
Member

bhagany commented Mar 15, 2012

Some sort of strangeness here - github won't let me automatically merge this. Kind of makes me nervous, but I don't immediately understand why, unless it's because there are commits that have already been made in the pul request?

@pydanny
Copy link
Copy Markdown

pydanny commented Mar 18, 2012

IMO, Github auto-merges should only be used on documentation changes, not for logic/functionality changes.

@pydanny
Copy link
Copy Markdown

pydanny commented Mar 19, 2012

Upon review, this should be merged in. I don't have rights, but it looks very solid to me.

@jjdelc
Copy link
Copy Markdown
Author

jjdelc commented Mar 19, 2012

Sorry guys, I've been offline during hte week, I'll go ahead and do a request without the duplicate commits.

@bhagany
Copy link
Copy Markdown
Member

bhagany commented Mar 19, 2012

Apologies here too, I've had a pretty busy half-week or so.

@jonas-grgt
Copy link
Copy Markdown

This should be merged in, why hasn't this been merged in yet ?

@bhagany
Copy link
Copy Markdown
Member

bhagany commented Oct 15, 2012

As you can see in the comments, I was left expecting an updated request without duplicate commits, and so dropped it.

@bhagany
Copy link
Copy Markdown
Member

bhagany commented Oct 15, 2012

The patch also removes simplejson as a dependency, and while I can sympathize with that goal, it has a few effects that I'm not sure I can support -

  • it wouldn't support older python versions
  • removes any hope for automatic Decimal serialization, which isn't there now, but I think it would be nice

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