Maryam Yusuf
Maryam Yusuf
> The following logic in `zulip_bots/zulip_bots/run.py` can use some more testing. In particular, it's not clear the intent behind the provision check in the `else` block. > > ```python >...
Hello @brylie, Can I take this?
Thank you, I will.
Hello @brylie, For this task, do you think it is ideal I add template tag {%trans%} to all the individual templates?
Hello @brylie I have created my first pull request for this task Link: https://github.com/CiviWiki/OpenCiviWiki/pull/976 I added translation to the landing page, header and footer If it is fine, I will...
Hello @brylie Thanks a lot for taking out time to review! I have accepted all the suggestions. It looks like you still need to merge the pull request, so it...
Thank you @brylie for taking out time to approve the pull request. Do you think there are other pages I need to add translation to?
@felixxm Thanks for the review, I have updated the changes requested.
> Thank you for this @mharyam this looks good > > My only issue with this is I feel it doesn't fit very well in `attribute:: InlineModelAdmin.classes`. I don't think...
@sarahboyce I have updated the changes. Please have a look when you have a chance. I also slightly updated the paragraph to make it more readable, but I am happy...