first of all thanks for your work improving Kevin's patch I look forward for SA I'm going to migrate a project i'm working on to this so I can test it fully.
So comments on your old post (I don't have one on the newer)
On 6/29/06, Charles Duffy <[EMAIL PROTECTED]> wrote:
actually the "inside the api" TG_* classes are deprecated, and the ones for SQLObject will be remove in a future version.
As for forcing people that's not correct, remenber we use quickstart for projects and that should take care of dropping the default identity model to each model.py so basically what needs to be done is clean all that and make sure http://trac.turbogears.org/turbogears/browser/trunk/turbogears/qstemplates/quickstart/%2Bpackage%2B/model.py_tmpl is updated, basically you need to drop your default model on the last part of the code and it's ready.
I believe what I said above covers it? or is there something else?
Charles Duffy wrote:
> My initial impression is positive, even though it's just been a quick
> glance so far. One issue: Identity and visit tracking haven't been
> updated to match. I'll try my hand at a patch in an hour or so if
> nobody's beat me to it.
Okay. Here are the issues I ran into:
- Defining default tables (inside visit.savisit and identity.saprovider)
with the same turbogears.database.metadata which users may later use to
define their *own* tg_user, tg_visit, etc. classes just doesn't work. I
solved this by simply taking the default tables/classes out of
identity.saprovider (since the templates should be providing their own
models for these, right?). That may not be the Right Way, though, since
it forces everyone to define their own identity classes even if they're
not changing anything; perhaps the configuration should be checked, and
these classes should be defined only if the identity.saprovider.model.*
configuration settings point to the classes in question?
actually the "inside the api" TG_* classes are deprecated, and the ones for SQLObject will be remove in a future version.
As for forcing people that's not correct, remenber we use quickstart for projects and that should take care of dropping the default identity model to each model.py so basically what needs to be done is clean all that and make sure http://trac.turbogears.org/turbogears/browser/trunk/turbogears/qstemplates/quickstart/%2Bpackage%2B/model.py_tmpl is updated, basically you need to drop your default model on the last part of the code and it's ready.
This patch does not yet attempt to make quickstarted SQLAlchemy apps
work out-of-the-box.
I believe what I said above covers it? or is there something else?
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "TurboGears" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at http://groups.google.com/group/turbogears
-~----------~----~----~----~------~----~------~--~---

