Dev

[Carobert85]

No description provided.

[mikeckennedy]

Hi @Carobert85 Was this a mistaken identity PR that was intended to be towards your fork or did you find a problem we should be looking at?

[Carobert85]

Good morning, Both! I forked the repo, and when I was trying to run the project, I had to fix a few things in the requirements.txt because more recent changes in some of the packages were incompatible with the code as it exists, and in the project files I had to handle some broken code in flask. I accidentally created the merge request onto your repo instead of mine and cancelled it when I realized my mistake. Happy to discuss in more detail if you’d like. Cheers, Charles

On Wed, Dec 28, 2022 at 10:06 AM Michael Kennedy ***@***.***> wrote: Hi @Carobert85 <https://github.com/Carobert85> Was this a mistaken identity PR that was intended to be towards your fork or did you find a problem we should be looking at? — Reply to this email directly, view it on GitHub <#3 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AKKAXNX5RQVQV5C5O7ZOHHDWPR6RTANCNFSM6AAAAAATKRUZSQ> . You are receiving this because you were mentioned.Message ID: ***@***.***>

-- Charles Mendelson 2064378073 3452 40th Ave SW, Seattle, WA

[mikeckennedy]

Thanks. I just retested every project (starter/final for each chapter) and couldn't find any issues (on Python 3.11).

But to be safe, I added pinned versions to every chapter for the requirements.txt:

fbaf023

[Carobert85]

Thank you! I’ll reinstall everything and check everything on my side. Hope you’re well, Charles

On Wed, Dec 28, 2022 at 10:34 AM Michael Kennedy ***@***.***> wrote: Thanks. I just retested every project (starter/final for each chapter) and couldn't find any issues (on Python 3.11). But to be safe, I added pinned versions to every chapter for the requirements.txt: fbaf023 <fbaf023> — Reply to this email directly, view it on GitHub <#3 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AKKAXNRMBVWI5E4JIKKDWLDWPSB2BANCNFSM6AAAAAATKRUZSQ> . You are receiving this because you were mentioned.Message ID: ***@***.***>

-- Charles Mendelson 2064378073 3452 40th Ave SW, Seattle, WA

[mikeckennedy]

Sounds good. You shouldn't need to reinstall things given the pinned versions now. Give pip install -r requirements.txt with the right venv active as a start.

[Carobert85]

👍

On Wed, Dec 28, 2022 at 10:37 AM Michael Kennedy ***@***.***> wrote: Sounds good. You shouldn't need to reinstall things given the pinned versions now. Give pip install -r requirements.txt with the right venv active as a start. — Reply to this email directly, view it on GitHub <#3 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AKKAXNRJLQYVXMSF7SQKGZDWPSCINANCNFSM6AAAAAATKRUZSQ> . You are receiving this because you were mentioned.Message ID: ***@***.***>

-- Charles Mendelson 2064378073 3452 40th Ave SW, Seattle, WA