Skip to content
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

Routing windowslivewriter manifest & XMLRPC MetaWeblog to support OpenLiveWriter #396

Conversation

warrenbuckley
Copy link
Contributor

@warrenbuckley warrenbuckley commented Jul 5, 2022

  • Adds the wlwmanifest/{id} route to return the XML used in conjunction with the RSD XML
  • Gets the XMLRPC / MetaWeblog Controller for receiving HTTP POSTed XML for OpenLiveWriter
  • Drops Cookcomputing.XMLRpc for WilderMinds.MetaWeblog
    https://github.com/shawnwildermuth/MetaWeblog

Help needed

I am unsure with DI to update the Root Articulate Blog Node ID - so appreciate any pointers @Shazwazza

private IPublishedContent BlogRoot()
{
    // TODO: Do not hardcode 1068 - need to pass it through from the controller route
    // When injecting this service into the controller - but how?!?
    var node = _umbracoContextAccessor.GetRequiredUmbracoContext().Content.GetById(1068);
    ...
}

Testing

  • Download Open Live Writer
  • Have test site running & put a breakpoint in MetaWeblogController to help with ddebugging
  • Add new configuration pointing to example site
  • Add a new post - via theme detection
  • It should delete the temp post & put it in the recycle bin
  • Add a new post
    • Select existing category
    • Add a new category
    • Select existing tags
    • Add a new tag
    • Add one or more image (Use the silly effects in LiveWriter)
    • Save draft & verify
    • Publish & verify
  • Close Open Live Writer
  • File -> Open Recent Post -> Select blog account from list (Fetches recent posts)
    • Select an item
    • Use delete button - ensure blog post is moved to recycle bin
    • Select an item to edit
    • Update content, properties, categories & tags - verify changes are expected

@Shazwazza
Copy link
Owner

Has some conflicts now. Then windows live writer integration will need to be tested

Warren Buckley added 2 commits July 6, 2022 10:18
@warrenbuckley
Copy link
Contributor Author

This is just the route and XML returned for the WLW Manifest
image

Will do a sperate PR for the route that is needed for WLW (Will also see if you can download/use it)

@warrenbuckley
Copy link
Contributor Author

OK not sure if Windows Live Writer is a thing anymore and the MetaWebBlog Spec is 404 from your comments
http://xmlrpc.com/metaWeblogApi.html

Is this something still worth supporting and migrating over @Shazwazza ?

@Shazwazza
Copy link
Owner

It's a thing, It's Open Live Writer now https://openlivewriter.com/

@warrenbuckley
Copy link
Contributor Author

Ok will have to take a look to try and figure out the rest of it.

Gotta love the XP/Windows Vista style vibes of the app still !

Warren Buckley added 2 commits July 7, 2022 12:03
@warrenbuckley
Copy link
Contributor Author

Making draft PR as will include the other routing stuff to get all of the MetaWeblog/Open Live Writer stuff working

@warrenbuckley
Copy link
Contributor Author

OK @Shazwazza I am gonna need your help here as I have fixed up the MetaWeblogHandler class that was commented out but I have no idea how to wire it up into the routes with RouteTransformer stuff you have.

Any advice or pointers you can give on how to fix/resolve this then I can try and attempt it

No Idea

@warrenbuckley warrenbuckley changed the title Routing/windowslivewriter manifest Routing windowslivewriter manifest & XMLRPC MetaWeblog to support OpenLiveWriter Aug 2, 2022
@warrenbuckley
Copy link
Contributor Author

warrenbuckley commented Aug 2, 2022

@Shazwazza ready for review & testing with Open Live Writer now :)

Ready when you are

src/Articulate/Controllers/MetaWeblogController.cs Outdated Show resolved Hide resolved
src/Articulate/Controllers/MetaWeblogController.cs Outdated Show resolved Hide resolved
src/Articulate/Components/ArticulateComposer.cs Outdated Show resolved Hide resolved
src/Articulate/Components/ArticulateComposer.cs Outdated Show resolved Hide resolved
src/Articulate/Controllers/MetaWeblogController.cs Outdated Show resolved Hide resolved
src/Articulate/MetaWeblog/ArticulateMetaWeblogProvider.cs Outdated Show resolved Hide resolved
src/Articulate/MetaWeblog/ArticulateMetaWeblogProvider.cs Outdated Show resolved Hide resolved
src/Articulate/MetaWeblog/ArticulateMetaWeblogProvider.cs Outdated Show resolved Hide resolved
src/Articulate/Routing/MetaWeblogHandler.cs Outdated Show resolved Hide resolved
# Conflicts:
#	src/Articulate/Articulate.csproj
#	src/Articulate/Models/PostModel.cs
#	src/Articulate/Routing/ArticulateRouter.cs
#	src/Articulate/Routing/MetaWeblogHandler.cs
@Shazwazza Shazwazza merged commit 45f83f0 into Shazwazza:release/5.0.0 Mar 9, 2023
@Shazwazza
Copy link
Owner

Thanks warren, i fixed a bunch up in your branch and merged 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants