Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Mention ZopeTransactionExtension and pyramid_tm in SQLAlchemy scaffold tutorial #749

Merged
merged 1 commit into from Jan 2, 2013

Conversation

Projects
None yet
3 participants
Contributor

irskep commented Dec 31, 2012

If this page is found via web search while trying to configure SQLAlchemy from scratch without a scaffold, it is helpful to mention pyramid_tm as a requirement somewhere. Otherwise, a hapless user such as your humble author might spend lots of time wondering why his sessions aren't getting committed after his requests despite his best efforts to properly configure SQLAlchemy and ZopeTransactionExtension.

Possible further improvements:

  • Dedicated section on configuring SQLAlchemy from scratch
  • Add ZopeTransactionExtension and pyramid_tm to the glossary so they can be made linkable
@irskep irskep Mention ZopeTxnExt and pyramid_tm in tutorial
If this page is found via web search while trying to configure
SQLAlchemy from scratch without a scaffold, it is helpful to
mention pyramid_tm as a requirement somewhere. Otherwise, a
hapless user such as your humble author might spend lots of
time wondering why his sessions aren't getting committed after
his requests despite his best efforts to properly configure
SQLAlchemy and ZopeTransactionExtension.
e91085e

I think such dependencies should instead be mentioned in Preparation section.

@mcdonc mcdonc added a commit that referenced this pull request Jan 2, 2013

@mcdonc mcdonc Merge pull request #749 from irskep/patch-1
Mention ZopeTransactionExtension and pyramid_tm in SQLAlchemy scaffold tutorial
b01f493

@mcdonc mcdonc merged commit b01f493 into Pylons:master Jan 2, 2013

Owner

mcdonc commented Jan 2, 2013

Seems reasonable, thank you! @tshepang : if you want to try to expand on it in Preparation, that'd be fine.

@irskep irskep deleted the irskep:patch-1 branch Jan 2, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment