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

global: move oauthclient into new module structure #13

Merged
merged 1 commit into from
Nov 25, 2015

Conversation

hachreak
Copy link
Member

No description provided.

@hachreak hachreak force-pushed the cookiecutter_template branch 2 times, most recently from fec53a3 to 8c3e07c Compare November 20, 2015 16:11
@jirikuncar jirikuncar added this to the v1.0.0 milestone Nov 20, 2015
@hachreak hachreak force-pushed the cookiecutter_template branch 7 times, most recently from 3b8f2aa to 32b8d65 Compare November 23, 2015 13:55
@hachreak
Copy link
Member Author

ping @lnielsen


def __init__(self, app=None):
"""Extension initialization."""
_('A translation string')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clean-up needed here

@hachreak hachreak force-pushed the cookiecutter_template branch 3 times, most recently from ff9a30f to cd644f4 Compare November 23, 2015 17:46
@lnielsen lnielsen removed their assignment Nov 24, 2015
@hachreak hachreak force-pushed the cookiecutter_template branch 3 times, most recently from 8f484a7 to 3b3d4e2 Compare November 25, 2015 10:43
InvenioAccounts(app)
FlaskOAuth(app)
InvenioOAuthClient(app)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

last 3 lines can be removed

@hachreak hachreak force-pushed the cookiecutter_template branch 3 times, most recently from 75d2199 to 686ca6a Compare November 25, 2015 13:28
@@ -104,28 +131,35 @@ def run_tests(self):
version=version,
description=__doc__,
long_description=readme + '\n\n' + history,
keywords='invenio oauth authentication',
keywords='invenio TODO',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

keywords='invenio oauth authentication',

@hachreak hachreak force-pushed the cookiecutter_template branch 7 times, most recently from 8870626 to f1f117f Compare November 25, 2015 15:21
db.session.commit()
with db.session.begin_nested():
self.access_token = token
self.secret = secret
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add db.session.add(self) in case somebody uses the function directly after initializing the object

* INCOMPATIBLE Updates module structure accordly with Invenio 3.
  (addresses inveniosoftware#12)

* Removes invenio-utils, invenio-ext, invenio-base, Flask-Registry
  dependencies.

* Refactor forms to use Flask-Security instead of custom
  implementation.

* Incorporates UserEXT table from invenio-accounts and renames it to
  UserIdentity.

* NOTE Disables extra fields loading from orcid provider.

Signed-off-by: Leonardo Rossi <leonardo.r@cern.ch>
@jirikuncar jirikuncar merged commit 827be00 into inveniosoftware:master Nov 25, 2015
@hachreak hachreak deleted the cookiecutter_template branch November 26, 2015 08:06
@lnielsen lnielsen mentioned this pull request Dec 1, 2015
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.

None yet

3 participants