-
Notifications
You must be signed in to change notification settings - Fork 68
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add optional UUID based base models #9
Comments
@nemesisdesign I'd like to do this next if it isn't reserved for GCI |
@R9295 this is not reserved to GCI, however, we are moving on different fronts at the same time. Anyway, before jumping on this, what about making the new passphrase feature available in openwisp-controller? openwisp/openwisp-controller#36 |
@nemesisdesign makes sense, will do 👍 |
Related to openwisp#9
All the OpenWISP 2 models have uuid based IDs except django-x509.
We need to have uuid based fields for django-x509 too, but since this app is also used by projects that are not related to openwisp, we could keep this feature optional.
Tasks:
BaseUUID
model which inherits BaseX509 which has a uuid field exactly as https://github.com/openwisp/openwisp-utils/blob/master/openwisp_utils/base.py#L13AbstractUUIDCa
andAbstractUUIDCert
, which inheritBaseUUID
AbstractUUIDCa
andAbstractUUIDCert
work as expected, (concrete models will have to be added as well)The text was updated successfully, but these errors were encountered: