Add URL to luxon text #176
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@cif, not sure how to run glide locally and use this branch to test that my change is working properly in the product. Any tips? |
|
For that change to render a link I suspect we'd have to convert the markdown in the string to HTML in the property configuration. If you create an issue for that I can look into it, might be a pretty simple change |
|
Created the issue here: https://github.com/glideapps/glide/issues/25182 I just added the URL to see if it would work. I saw that there was a URL previously stored in ScreenshotswithAbout withAbout type |
|
Commented in the issue as well https://github.com/glideapps/glide/issues/25182#issuecomment-1910436021. Adding the markdown to the description field will be the way we want to enable this - the builder ignores the about field for yes code configs |
There was a problem hiding this comment.
Approving ahead of https://github.com/glideapps/glide/pull/25188 needing to land in production, which I believe should happen today.
|
Nice! Do you want to merge this or should I once the other PR lands? |
|
Looks like the prod deploy hasn't started yet and I'll be offline for a few hours. If you see it land feel to merge! |
It landed! I would merge, but I'm not authorized apparently. Can you give me permissions? |
|
I also lack permissions here, @timwellswa can you give us a hand when you get a second? |
Context
Make it easier to understand luxon date formatting by adding a hyperlink
Screenshot