Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Donation tab update #356

Merged
merged 4 commits into from
Jan 17, 2024
Merged

Conversation

arcataroger
Copy link
Contributor

@arcataroger arcataroger commented Jan 7, 2024

Updated the donation tab per freelawproject/recap#359

image

The link goes to https://donate.free.law/forms/supportflp

@mlissner
Copy link
Member

mlissner commented Jan 8, 2024

Thanks, this is great! Can you also change the tab name to say "Donate" and make the button say "Donate to Free.law"?

@arcataroger
Copy link
Contributor Author

Thanks, this is great! Can you also change the tab name to say "Donate" and make the button say "Donate to Free.law"?

Sure, changed!

image

@mlissner
Copy link
Member

mlissner commented Jan 8, 2024

I turned on the checks for you, and you got busted by the Release Notes bot. Can you add something, please?

@arcataroger
Copy link
Contributor Author

arcataroger commented Jan 9, 2024

Sure, I'll prep it as though it were a new version then (sorry, wasn't sure if you had another developer who normally did that). Give me a day or two :)

This means that every PR is going to be its own new version, is that OK? Or did you want to try to roll several smaller fixes into one release? (I'm happy to work however is best for you all... just let me know the proper procedure!)

@ERosendo
Copy link
Contributor

@mlissner This code looks good and I've updated the CHANGES.md file.

did you want to try to roll several smaller fixes into one release?

@arcataroger We typically bundle minor fixes or updates into a single release.

@mlissner mlissner merged commit beea4cd into freelawproject:main Jan 17, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants