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

feature: href for logo (with optional target="_blank") #171

Merged
merged 10 commits into from
Sep 17, 2024

Conversation

breftejk
Copy link
Contributor

@breftejk breftejk commented Aug 8, 2024

Checklist

@breftejk breftejk changed the title [Feature] href for logo (with optional target="_blank") feature: href for logo (with optional target="_blank") Aug 8, 2024
@mcollina
Copy link
Member

mcollina commented Aug 9, 2024

the tests are failing

Uzlopak
Uzlopak previously requested changes Aug 9, 2024
Copy link
Collaborator

@Uzlopak Uzlopak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would make it actually simpler

types/index.d.ts Outdated Show resolved Hide resolved
lib/swagger-initializer.js Outdated Show resolved Hide resolved
lib/swagger-initializer.js Outdated Show resolved Hide resolved
breftejk and others added 2 commits August 19, 2024 10:55
Co-authored-by: Aras Abbasi <aras.abbasi@googlemail.com>
Signed-off-by: Marcin Kondrat <marcin.kondrat@assts.tech>
Co-authored-by: Aras Abbasi <aras.abbasi@googlemail.com>
Signed-off-by: Marcin Kondrat <marcin.kondrat@assts.tech>
README.md Outdated Show resolved Hide resolved
@breftejk
Copy link
Contributor Author

breftejk commented Aug 21, 2024

What is with this test error? I dont get it. Everything's fine on my test yet still this weird error on GitHub.

It doesn't even tell you the reason to be backtracked.

@climba03003
Copy link
Member

Ignore the pnpm one, but e2e must be pass.

@Uzlopak Uzlopak dismissed their stale review September 17, 2024 02:50

i modified it

@Uzlopak
Copy link
Collaborator

Uzlopak commented Sep 17, 2024

@climba03003 PTAL

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Uzlopak Uzlopak merged commit 3e5850b into fastify:master Sep 17, 2024
16 checks passed
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.

4 participants