plugindemo icon indicating copy to clipboard operation
plugindemo copied to clipboard

Add all fields for example manifest file

Open FlowingSPDG opened this issue 1 year ago • 3 comments

Hello, thanks for awesome plugin template! I am developing some small plugin for rewriting [GET]URL query to [POST]JSON body. (This is helpful for emulating AWS API Gateway w/o localstack)

I was trying to build my plugin but I was struggling with import error. https://github.com/FlowingSPDG/traefik-plugin-query-to-json/issues/6

And I digged into piceus's source code, and found out there's a basePkg field which is not listed unlike import field. https://github.com/traefik/piceus/blob/909a0aa078ceb53ca4af6dd4b299c4efc0b0335e/pkg/core/types.go#L16-L28

(Also import field automatically replaces hypen(-) to underscore(_) so I think that's why I was struggling) https://github.com/traefik/piceus/blob/909a0aa078ceb53ca4af6dd4b299c4efc0b0335e/pkg/core/yaegi.go#L153-L158

So can we add document for import/basePkg field, and note that import field value will be replaced on Yeagi check process?

Thank you.

FlowingSPDG avatar Sep 11 '24 07:09 FlowingSPDG

Hey @FlowingSPDG,

Thanks for your suggestion, we think it makes a lot of sense.

Unfortunately, this would not make it to our roadmap for a while as we are focused elsewhere.

If you or another community member would like to build it, let us know, and we will work with you to make sure you have all the information needed so that it can be merged.

nmengin avatar Oct 10 '24 13:10 nmengin

Hi again @nmengin !

It is totally okay, and thank you for reviewing my suggestion :)

I'll be waiting for you if you need any help, I love traefik ❤

FlowingSPDG avatar Oct 10 '24 15:10 FlowingSPDG

So can we add document for import/basePkg field, and note that import field value will be replaced on Yeagi check process?

We agree with your suggestion to improve the documentation. So so not hesitate to ping me on any PR you could open on this topic.

I'll be waiting for you if you need any help, I love traefik ❤

Thank you 😄

nmengin avatar Oct 10 '24 16:10 nmengin