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.

Sign up for GitHub

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

Jump to bottom

Number of Middlewares displayed in OpenAPI must be parametrable #350

Open
EwenQuim opened this issue Jan 13, 2025 · 24 comments
Open

Number of Middlewares displayed in OpenAPI must be parametrable #350

EwenQuim opened this issue Jan 13, 2025 · 24 comments
Assignees
@Michaelkingsdev
Labels
good first issue Good for newcomers OD Boost Issues for the OnlyDust event
Milestone
v0.19

Comments

@EwenQuim
Copy link
Member

Want we want

Users might want to see 10 middlewares, or only the first 3 if they want, with a default of 5.

It must be a Engine.OpenAPIConfig option that users can set with WithOpenAPIConfig.

Please don't forget tests! 🔥

Example of current behaviour

image

👉🏼 Currently it is not configurable
@EwenQuim EwenQuim added the good first issue Good for newcomers label Jan 13, 2025
@olisaagbafor
Copy link
Contributor

olisaagbafor commented Jan 13, 2025 •
edited
Loading

@EwenQuim May I pick this up?

@EwenQuim
Copy link
Member Author

As you already have an issue assigned, I leave this others, but once it's done, you can work on this one!

@olisaagbafor
Copy link
Contributor

@EwenQuim I have done the other issue, can I now get into this?

@EwenQuim
Copy link
Member Author

As long as the other issue isn't close and the PR isn't merged, I don't consider it Done ;)

@zachwood222
Copy link

Can I handle this task?

@EwenQuim
Copy link
Member Author

Yes @zachwood222 , no problem!

@aidenwong812
Copy link

Could I be assigned to this?

@EwenQuim
Copy link
Member Author

Yes no problem! Are you giving up on #285 ?

@olisaagbafor
Copy link
Contributor

@EwenQuim am waiting patiently...

@Elite-tch
Copy link

Could I take a shot at this?

@ryzen-xp
Copy link

Can I jump on this task?

@EwenQuim
Copy link
Member Author

@Elite-tch @ryzen-xp why not, you can try!

@ekumamatthew
Copy link
Contributor

Can I tackle this one?

@EwenQuim EwenQuim added the OD Boost Issues for the OnlyDust event label Jan 20, 2025
@chiscookeke11
Copy link

Could I take over this issue?

@0xprinc
Copy link

0xprinc commented Jan 22, 2025

Could I try solving this?

@Supa-mega
Copy link

Can I handle this task?

@Elite-tch
Copy link

Elite-tch commented Jan 22, 2025 via email

@EwenQuim
Copy link
Member Author

EwenQuim commented Jan 22, 2025 •
edited
Loading

Thank you for your enthusiasm. :)

You're already 5 on this issue, please try some other issues!

We need help on #361, #366, #285, #356, #357 if you want 🙏🏼

@Michaelkingsdev
Copy link

I’d love to work on this task.

@demola234
Copy link

May I take this issue on?

@abdegenius
Copy link

Can I start working on this?

@ShantelPeters
Copy link

Can I take this issue?

@EwenQuim
Copy link
Member Author

If everybody on this thread stars Fuego, we might soon reach 1200 stars, please do it!

@EwenQuim
Copy link
Member Author

EwenQuim commented Feb 1, 2025

@Michaelkingsdev still interested?

@EwenQuim EwenQuim added this to the v0.19 milestone Feb 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Michaelkingsdev Michaelkingsdev

Labels
good first issue Good for newcomers OD Boost Issues for the OnlyDust event
Projects
None yet
Milestone
v0.19
Development

No branches or pull requests