Conversation
yashikakhurana
left a comment
There was a problem hiding this comment.
ahaa I see adding a subscriber for that, just a suggestion the name subscriber is little bit confusing for me, as we have subscriber for the experiments too, seeing the same field might be little confusing 🤔
Its the same name on two different models. And the concept is the same -- a subscription. We have types, I don't see this being a problem. |
| @@ -0,0 +1,20 @@ | |||
| # Generated by Django 5.2.7 on 2025-10-27 15:00 | |||
There was a problem hiding this comment.
This needs a ruff format.
There was a problem hiding this comment.
We must have a bug on our make lint step as it sometimes doesn't report these.
7e9e33e to
022334d
Compare
yashikakhurana
left a comment
There was a problem hiding this comment.
yeah I understand the concept is same, I said it is confusing for me :D, lets ![]()
Because
This commit
Fixes #13783