ojhaujjwal
Repos
129
Followers
55
Following
90

Exchange rates/Currency Converter Library for PHP

48
22

Like laradock but for Magento instead of Laravel.

28
7

capistrano-presentation

0
0

Image manipulation module for Zend Framework

16
15

paale-dai is an express based middleware for creating SSO based authentication microservice.

Alternative to PHP's native session handler for PSR-7.

7
2

Events

pull request opened
Handle async errors by passing the error to next middleware

Check List

  • [x] Tests has been run in packages where changes made if available
  • [x] Linter has been run for changed code
  • [x] Tests for the changes have been added if not covered yet

Description of Changes Made (if issue reference is not provided) If express route handlers/middleware return async errors/rejected promise, they aren't passed on to next automatically but it's passed on for sync errors/uncaught exceptions.

I believe this is changing as a part of express 5 but I think this is a worthwhile addition until express 5 final release is out and we migrate to express 5.

Created at 2 weeks ago
create branch
ojhaujjwal create branch feature/handle-error
Created at 2 weeks ago
started
Created at 2 weeks ago
issue comment
Rename .d.ts files to .ts files in order to expose the types in the build

This is a very small change.

Can someone please review or merge this if all's okay? Not a fan of using any types in typescript.

Created at 3 weeks ago
pull request opened
Rename .d.ts files to .ts files in order to expose the types in the uild

Check List

  • [x] Tests has been run in packages where changes made if available
  • [x] Linter has been run for changed code
  • [x] Tests for the changes have been added if not covered yet
  • [x] Docs have been added / updated if required

I imported QueryFilter from the api-gateway package but the typing didn't work. For example:

The type exists for CreateOptions, so TS is giving type errors but not for QueryFilter.

I looked in the imported package folder in my project and non of the .d.ts files were there.

Only the enum.d.ts was there because it's a .ts file in the source already.

This should help the TS experience.

Created at 1 month ago
create branch
ojhaujjwal create branch rename-d-ts-to-ts
Created at 1 month ago
Created at 1 month ago
closed issue
MergIng the critical and high vulnerability dependencies

Thank you for this awesome tool.

Problem Just doing a quick yarn audit in the repo, I got this:

I am slightly concerned by the number of high and critical issues. I am not complaining since this is an open source tool and this is freaking awesome. I am only opening the issue there because I am trying to use this and I have just been asked questions regarding the security aspect of the repo.

Is there something we can do about update security vulnerabilities or something where you guys need help to maintain?

Created at 1 month ago
issue comment
MergIng the critical and high vulnerability dependencies

Thanks for the reply @paveltiunov. Closing the issue for now.

Created at 1 month ago
opened issue
MergIng the critical and high vulnerability dependencies

Thank you for this awesome tool.

Problem Just doing a quick yarn audit in the repo, I got this:

I am slightly concerned by the number of high and critical issues. I am not complaining since this is an open source tool and this is freaking awesome. I am only opening the issue there because I am trying to use this and I have just been asked questions regarding the security aspect of the repo.

Is there something we can do about update security vulnerabilities or something where you guys need help to maintain?

Created at 1 month ago