DPP icon indicating copy to clipboard operation
DPP copied to clipboard

User-installed commands

Open raxyte opened this issue 1 year ago • 6 comments

Adds required support for user-installed commands.

Code change checklist

  • [x] I have ensured that all methods and functions are fully documented using doxygen style comments.
  • [x] My code follows the coding style guide.
  • [ ] I tested that my change works before raising the PR.
  • [x] I have ensured that I did not break any existing API calls.
  • [x] I have not built my pull request using AI, a static analysis tool or similar without any human oversight.

raxyte avatar Aug 12 '24 21:08 raxyte

Deploy Preview for dpp-dev ready!

Name Link
Latest commit 09e8e0236561682244ef334ddc81b5b903214d8d
Latest deploy log https://app.netlify.com/sites/dpp-dev/deploys/671674b8bdab330008f9658e
Deploy Preview https://deploy-preview-1223--dpp-dev.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Aug 12 '24 21:08 netlify[bot]

I believe you should cover this feature with unit tests. (Look how other "online" tests are made for similar things)

ruslan-ilesik avatar Aug 12 '24 22:08 ruslan-ilesik

Try using DPP_DEPRECATED instead of [[deprecated]] to fix those errors on windows

Mishura4 avatar Aug 30 '24 00:08 Mishura4

@raxyte any movement on this pr, or is it dead?

braindigitalis avatar Oct 08 '24 16:10 braindigitalis

Don't have time to get back to it. Can close if no one else can take over it.

raxyte avatar Oct 08 '24 17:10 raxyte

I can try take over :)

Jaskowicz1 avatar Oct 08 '24 17:10 Jaskowicz1

if theres no movement on this, i'm happy to take it on?

braindigitalis avatar Oct 21 '24 15:10 braindigitalis

Please do, whilst I want to do it, I do not have the time :)

On Mon, 21 Oct 2024 at 16:15, Craig Edwards (Brain) < @.***> wrote:

if theres no movement on this, i'm happy to take it on?

— Reply to this email directly, view it on GitHub https://github.com/brainboxdotcc/DPP/pull/1223#issuecomment-2426978816, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALF2B6KGYVVTEW6DLFYKHDLZ4UK7RAVCNFSM6AAAAABMNAABZ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRWHE3TQOBRGY . You are receiving this because you commented.Message ID: @.***>

Jaskowicz1 avatar Oct 21 '24 15:10 Jaskowicz1

i have merged this into a branch on the dpp main repo called userapps and will continue to work on it. This has been done without rebase, so that credit for existing work is preserved.

braindigitalis avatar Oct 21 '24 15:10 braindigitalis