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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃搮 Add Last Seen On as a system field so it can be used in campaigns #2902
Conversation
KEY_NAME: dict(label="Name", value_type=Value.TYPE_TEXT), | ||
KEY_CREATED_ON: dict(label="Created On", value_type=Value.TYPE_DATETIME), | ||
KEY_LANGUAGE: dict(label="Language", value_type=Value.TYPE_TEXT), | ||
KEY_LAST_SEEN_ON: dict(label="Last Seen On", value_type=Value.TYPE_DATETIME), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think these labels are supposed to be localized in the DB.. we could find a way to localize them in the UI.. tho even that could get confusing since Name
will be always be name
in expressions etc
@@ -1094,45 +1094,6 @@ def create_welcome_topup(self, topup_size=None): | |||
return TopUp.create(self.created_by, price=0, credits=topup_size, org=self) | |||
return None | |||
|
|||
def create_system_groups(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a little bit of dependency re-arranging - Org
shouldn't need to know how fields and groups are created
Codecov Report
@@ Coverage Diff @@
## master #2902 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 378 379 +1
Lines 25429 25441 +12
=========================================
+ Hits 25429 25441 +12
Continue to review full report at Codecov.
|
No description provided.