Skip to content

Conversation

@Aperrix
Copy link
Contributor

@Aperrix Aperrix commented Dec 18, 2020

  • Add fastify-blipp to display availables routes in dev env
  • Fix response typo (from res.send to return because req is async)
  • Fix server start function

@Aperrix Aperrix changed the title fix : improvements fix: improvements Dec 18, 2020
@2color 2color self-requested a review January 13, 2021 11:49
Copy link
Contributor

@2color 2color left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @Aperrix.

For simplicity, I'd remove the fastify-blipp plugin.

I left a couple of small comments.

Finally, be sure to run prettier for consistent code style.

@janpio
Copy link
Contributor

janpio commented Feb 16, 2021

Can you take another look at this @Aperrix? Thanks.

@janpio janpio changed the title fix: improvements fix(javascript/rest-fastify): improvements Feb 18, 2021
@ruheni ruheni closed this in #4233 Nov 7, 2022
@ruheni
Copy link
Contributor

ruheni commented Nov 7, 2022

Closed this in favor of #4233

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