activist-org / activist

An open-source activism platform
https://activist.org
GNU Affero General Public License v3.0
209 stars 173 forks source link

Tests for the password reset endpoints #907

Closed thesarfo closed 1 week ago

thesarfo commented 2 weeks ago

Contributor checklist


Description

This pull request proposes to add comprehensive tests for the password reset functionality in the authentication app. It aims to ensure that the password reset process works correctly and handles various scenarios properly.

Changes:

github-actions[bot] commented 2 weeks ago

Thank you for the pull request!

The activist team will do our best to address your contribution as soon as we can. The following is a checklist for maintainers to make sure this process goes as well as possible. Feel free to address the points below yourself in further commits if you realize that actions are needed :)

If you're not already a member of our public Matrix community, please consider joining! We'd suggest using Element as your Matrix client, and definitely join the General and Development rooms once you're in. Also consider joining our bi-weekly Saturday dev syncs. It'd be great to have you!

Maintainer checklist

netlify[bot] commented 2 weeks ago

Deploy Preview for activist-org canceled.

Name Link
Latest commit e353594b77e389657071aea7ede70e2a6222654c
Latest deploy log https://app.netlify.com/sites/activist-org/deploys/6680590dc2f9740008b01612
andrewtavis commented 2 weeks ago

Thanks for sending this along, @thesarfo! We'll check it and get back with feedback as soon as we can :)

to-sta commented 1 week ago

@thesarfo thanks for your PR ๐Ÿš€ . I fixed a few minor details and ran the tests successfully.

andrewtavis commented 1 week ago

Thanks for the work here, @thesarfo! Give me just a moment to send along some changes, and then I'll take care of any merge conflicts in here. Should be in by the start of the weekend ๐Ÿ˜Š

thesarfo commented 1 week ago

@andrewtavis anytime! And thanks for the opportunity๐Ÿ˜Š