astro icon indicating copy to clipboard operation
astro copied to clipboard

Django 2.2 needs mysqlclient >= 1.3.13

Open bahoo opened this issue 5 years ago • 3 comments

Hello! Django 2.2 requires mysqlclient>=1.3.13 per this commit. lambda-packages currently offers mysqlclient at 1.3.12. I'd like to raise the issue and request an upgrade.

Django 2.2 was released earlier today — presumably, not on an April Fool's basis 😅— and has been designated a Long Term Support release, which will receive security updates for at least three years into the future, so I think it's worth the effort.

Compiling it myself is a bit outside of my typical wheelhouse, so I'm happy to defer to others :D but otherwise, I'll take a crack at it this evening

( Not clear to me, also, if there might be other work necessary to get this update into Zappa. )

bahoo avatar Apr 02 '19 00:04 bahoo

There is a pull request related to this: #95

tbaxx avatar Apr 04 '19 06:04 tbaxx

Unless this library is being retired, this issue really should be addressed pretty quick.

celeduc avatar May 04 '19 23:05 celeduc

See also #97 #95 #100

celeduc avatar May 07 '19 09:05 celeduc

Thanks! This is a legit bug I think. I can take a look as I'm currently in the middle of working on build stuff.

matthewp avatar Apr 15 '21 18:04 matthewp

Ok, I've had some time to think about this and I don't think it's a bug, but it is definitely an interesting discussion. As you said, I could see people wanting to have files in their astro/ folder and reference them as being relative to the currently astro page's file. But at the same time, people will definitely want to reference relative to the current page URL.

I'd like to hold off on doing anything on this for now and see what other people think; if any sort of pattern emerges out of this. For example, I could see an argument for considering images inside of the astro/ folder as being processed differently, like perhaps they would be base64 inlined in this case.

I think the best way to experiment with these ideas for now is to create a custom component that encapsulates this idea. For example you might have:

<Image src="./social.png" inlined />

Where the Image component makes this internally referenced to the astro folder and in this case inlines the image.

matthewp avatar Apr 19 '21 12:04 matthewp

Ok, I've had some time to think about this and I don't think it's a bug, but it is definitely an interesting discussion.

I did make some changes to how files within the Astro folder (now src/) were resolved, and this should be either fixed or at least possible now, even with collections and dynamically-generated pages. Every file in src/ should now be represented 1:1 in the final /_astro/ output folder, so we should just handle this for users.

While I think public/ is a good pattern, it can also become messy having only one place to put static assets. I’d like to let the user decide what’s best for them, and sometimes having assets live alongside pages is more beneficial.

drwpow avatar Jun 14 '21 16:06 drwpow

We are tracking a fix for this in this RFC: #310, so closing this one.

matthewp avatar Jul 27 '21 16:07 matthewp