Notifications for when someone posted a comment on your node
was not created.
Root cause was that default values defined in schema was not set,
resulting in activity subscriptions not being active.
There were 2 bugs preventing them to be set:
* The way the caching of markdown as html was implemented caused
default values not to be set.
* Eve/Cerberus regression causes nested default values to fail
https://github.com/pyeve/eve/issues/1174
Also, a 3rd bug caused nodes without a parent not to have a
subscription.
Migration scripts:
How markdown fields is cached has changed, and unused properties
of attachments has been removed.
./manage.py maintenance replace_pillar_node_type_schemas
Set the default values of activities-subscription
./manage.py maintenance fix_missing_activities_subscription_defaults
Part of the code assumed shortcodes were globally unique, and another part
assumed the shortcodes are unique per project (the latter is correct).
Now the project ID is taken from the URL the Subversion hook pushes to.
Note that this looks at the value directly; when using {'key': None} as
value, it is not None and thus will be set. This thus is NOT a way to
be 100% sure there are no None values.
The Eve hook that added 'parent_info' caused this issue, as the validation
fails when saving a document that contains this. The easiest fix was to
prefix it with an underscore, so that the SDK automatically strips it
before saving.
If revisions 1-130 exist, it errors at "svn log -r 1:250", so we can't
reliably give a last number. Easiest solution is to just forego per-batch
processing, and fetch all the unseen entries in one go.
It isn't triggered by anything yet. When the observer is called, it uses
Blinker to send out a signal for every [T12345] marker it sees in the
first line of each commit log. Those signals aren't connected to anything
yet.
NOTE: this requires the 'svn' Python module , which is a wrapper for the
'svn' commandline client. This client needs to be installed on our docker
when we deploy.