Fix start of circuits with "D" problems

boolder-org / boolder-ios

Boolder iOS app https://www.boolder.com/en/app

MIT License

11 stars 3 forks source link

Fix start of circuits with "D" problems #20

Closed mirrom closed 6 days ago

mirrom commented 1 year ago

Fixes #15. This is a pretty simple fix without introducing new logic. The only downside I see is that there are two queries to the database to get the first problem if the circuit does not have a "D" problem ...

nmondollot commented 1 year ago

Thank you very much for your contribution! 🙏

I made a few tests, it works great 👍 Don't worry about the 2 sql queries, performance is really not an issue in this case.

I'm also adding a few comments to improve the code, happy to discuss! Let met know when you're ready for me to merge the PR.

Big thanks!