<div dir="ltr">Sorry about the hanging sentence fragment!  I forgot to delete it while drafting.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Feb 1, 2019 at 11:09 AM Daniel Alley <<a href="mailto:dalley@redhat.com">dalley@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div>Excellent writeup Tanya!</div><div><br></div><div>There is another change to be aware of as well, which I will mention here: "ContentSerializer" has been removed and replaced with more specialized versions -- "NoArtifactContentSerializer", "SingleArtifactContentSerializer", and "MultipleArtifactContentSerializer".  You now choose which Serializer to subclass based on the needs of your specific content type.  The idea behind this change is to reduce boilerplate for the former two use cases, which are both more common than the case of having multiple artifacts associated with one content object.</div><div><br></div>This diff from the pulp_file plugin is a good demonstration of what changes are necessary: <a href="https://github.com/pulp/pulp_file/pull/159/files" target="_blank">https://github.com/pulp/pulp_file/pull/159/files</a><br><div><div><br></div><div>This reduces <br></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Feb 1, 2019 at 10:27 AM Tatiana Tereshchenko <<a href="mailto:ttereshc@redhat.com" target="_blank">ttereshc@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Hi all,</div><div><br></div><div>There was a thread here about namespacing plugins + the idea of dropping `pulp_` from the app label. <br></div><div>Those changes [0] are merged now and some plugins have already been updated.</div><div><br></div><div>It is not necessary but it's encouraged to update the label of your plugin - to drop 'pulp_' part of it if you followed that convention.</div><div>If you intend to change your app label, do so before you start committing migrations since the app label is used to name tables in the DB. <br></div><div>Even if you hesitant to change the label, take a look at the changes in viewsets [1], you'd probably like to adjust endpoint_name anyway.<br></div><div> See pulp_file PR[1] as an example of changes to be made (the __init__.py and changes in viewsets are most important, the rest are docs mostly).<br></div></div><div dir="ltr"><br></div><div>Useful for any dev:<br></div><div>If you use one of the plugins which changed their app label in your dev environment, you need to recreate plugin migrations after app label is changed. <br></div><div>E.g. for pulp_file:<br></div><div> remove everything from the migrations directory of the plugin, e.g. rm -rf  pulp_file/app/migrations/<br></div><div> pulp-manager makemigrations <b>file</b></div><div> pulp-manager migrate<br></div><div dir="ltr"><div><br></div><div>[0] <a href="https://pulp.plan.io/issues/4279#note-12" target="_blank">https://pulp.plan.io/issues/4279#note-12</a></div><div>[1] <a href="https://github.com/pulp/pulp_file/pull/157/" target="_blank">https://github.com/pulp/pulp_file/pull/157/</a><br></div></div></div></div></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>
</blockquote></div>