- Issue created by @MegaKeegMan
- πΊπΈUnited States MegaKeegMan
While there is certainly more to do here, I have successfully posted a message via this plugin implementation in the issue fork. The post here currently only contains a message. Before doing more, I need to review and consider building the post data from the SPC side.
- πΊπΈUnited States MegaKeegMan
Issue title was updated to reflect architectural change, where the plugin definition was moved from Social Post Controller (which was renamed and moved to MPOSSE), and moved into its dependency https://www.drupal.org/project/drutopia_microblog β . The issue fork will need to be updated to reflect the namespacing of the extended base plugin, just a small change.
- πΊπΈUnited States freelock Seattle
Hi,
Plugin looks basically fine, but should use dependency injection to get the bsky.post_service service on line 24...
Should be along the lines of implementing the ContainerFactoryPluginInterface with a create() method and a constructor...
Cheers,
John - πΊπΈUnited States MegaKeegMan
Thanks for the feedback, I will try that out
- πΊπΈUnited States MegaKeegMan
Alright I did update, haven't tested yet. Currently trying to figure out why my IDE is upset about the create function not being compatible with ContainerFactoryPluginInterface
- πΊπΈUnited States MegaKeegMan
Alright was simply using the wrong use statement for ContainerInterface. Will let you know once I have tested this again
- πΊπΈUnited States MegaKeegMan
I have tested the code, and it is working okay. I'm not yet ready to add facets or images yet, as I have work to do on the drutopia microblog side and decisions to make. But it is functional and useful as it is now, too.