Closed HelloYeew closed 2 years ago
Hey @HelloYeew, thank you so much for this PR! I'm really sorry for not noticing your contribution until now, I haven't had much time to keep track of things outside work or uni 🙇♀️
This is a great addition, I'll merge your code and create a new version (I see you used 1.3.1, I'll do the same), thanks! 🥰 I'm really happy to see you using this workflow in your profile, by the way. I hope you find it useful.
Hi! I use your workflow for a while now and I think that it's better if someone curious that what's the album and currently when we click on the album it will send you to the image file.
So I think that why we just add the link to the last.fm page on that album and when someone curious what's album there so just click that album.
New parameter
To make this as an optional option I add
INCLUDE_LINK
parameter to the setup. It's optional and the default value is false so if you want to enable the link just addINCLUDE_LINK: true
to the setup. I have updated setup detail on readme.Result sample
I release the tag on my fork and try it on my profile repository so you can see the sample result from here. The visual is the same as before add the link.
About the code, I test on
INCLUDE_LINK
value and it's return astrue
orfalse
as string so I need to make the condition asos.getenv('INCLUDE_LINK') == 'false'
.