search

btcpayserver / btcpayserver-doc

BTCPay Server Official Documentation
https://docs.btcpayserver.org
379 stars 256 forks source link

Include plugins in nav as external #1410

Closed TChukwuleta closed 2 months ago

TChukwuleta commented 3 months ago

@ndeet included plugins documentations for some other plugins as external links in the plugin navs

dennisreimann commented 3 months ago

Why not integrate them directly, like we did with the other ones?

TChukwuleta commented 3 months ago

yea.. I think that'd be cooler.

I'd update it

ndeet commented 3 months ago

Yes that would also work, then add it to Plugins section in the sidebar like this

TChukwuleta commented 3 months ago

made the update @dennisreimann @ndeet

You can take another look

dennisreimann commented 3 months ago

Very good! Only thing I saw: There seems to be something funky going on with the first headline of the SideShift and Dynamic Reports plugins. Their source looks fine though.

ndeet commented 3 months ago

Yes, those two headlines seem to have some formatting issue, maybe some whitespace in front or at the end of the headlines?

I would also add rockstardev's Payroll plugin and Nicolas Boltcard plugin

TChukwuleta commented 3 months ago

Sideshift and Dynamic reports are fine now. I removed unprintable characters from readme

I'd add the payroll and the boltcard plugin

TChukwuleta commented 2 months ago

Hey @ndeet I think we can merge this for now. There is an issue with Boltcard ReadMe file. Tried reformatting and all but not still working but other ReadMe file works

I would probably open a PR on the bolt card plugin. But I believe we are good to go on the others including payroll plugin

pavlenex commented 2 months ago

LGTM. Great improvements @TChukwuleta 🎉 Some of these require cleanups (the readme's) but that's unrelated to this PR.