search

ucladevx / Bmaps-Backend

Documentation and source code for public events/locations APIs.
http://github.com/ucladevx/Bmaps-Backend/
MIT License
6 stars 2 forks source link

nltk_data / werkzeug error - cannot import 'url-decode' #172

Closed insiyab closed 4 years ago

insiyab commented 4 years ago

Running into this error when I run make dev on backend. (I made sure to run make build-base beforehand).

api_1       | [nltk_data]   Package punkt is already up-to-date!
api_1       | Traceback (most recent call last):
api_1       |   File "app.py", line 15, in <module>
api_1       |     from mappening import app, db
api_1       |   File "/app/mappening/__init__.py", line 9, in <module>
api_1       |     from mappening.auth.auth import auth
api_1       |   File "/app/mappening/auth/auth.py", line 2, in <module>
api_1       |     from mappening.api.models.user import User
api_1       |   File "/app/mappening/api/models/user.py", line 1, in <module>
api_1       |     from mappening.auth.facebook import facebook_oauth
api_1       |   File "/app/mappening/auth/facebook.py", line 3, in <module>
api_1       |     from flask_oauth import OAuth
api_1       |   File "/usr/local/lib/python3.7/site-packages/flask_oauth.py", line 19, in <module>
api_1       |     from werkzeug import url_decode, url_encode, url_quote, \
api_1       | ImportError: cannot import name 'url_decode' from 'werkzeug' (/usr/local/lib/python3.7/site-packages/werkzeug/__init__.py)
insiyab commented 4 years ago

Here's a potential fix that I found: https://github.com/scoringengine/scoringengine/issues/670 (change werkzeug.url_decode to werkzeug.urls.url_decode or pin Werkzeug to an earlier version < 1.0)

But I'm not sure in which file I should actually apply the fix to.

wfehrnstrom commented 4 years ago

Oh you know what this is actually stemming from files that will be deleted as part of our spring cleaning, so we don't really need to fix this issue

Kimeiga commented 4 years ago

oh yeet

Hakan Alpay Computer Science, UCLA

On Fri, Apr 24, 2020 at 6:03 PM Will Fehrnstrom notifications@github.com wrote:

Oh you know what this is actually stemming from files that will be deleted as part of our spring cleaning, so we don't really need to fix this issue

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ucladevx/Bmaps-Backend/issues/172#issuecomment-619295533, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE5CFT4RVOB3ALH5XRQLGGDROIZGBANCNFSM4MPGUS2Q .

insiyab commented 4 years ago

Made a temporary patch to fix this until werkzeug is removed completely: https://github.com/ucladevx/Bmaps-Backend/pull/174