Skip to content

Fix: Resolve h is undefined error#1

Open
ogoyon-coffreo wants to merge 5 commits intomasterfrom
feature/post-body_compilation
Open

Fix: Resolve h is undefined error#1
ogoyon-coffreo wants to merge 5 commits intomasterfrom
feature/post-body_compilation

Conversation

@ogoyon-coffreo
Copy link
Copy Markdown
Collaborator

@ogoyon-coffreo ogoyon-coffreo commented Feb 4, 2021

update the official package to compile after minification

@emri99
Copy link
Copy Markdown
Member

emri99 commented Feb 4, 2021

Can you add a description of what feature is added through this PR ?
Why is the build modified ?

It's difficult to perform a valueable review without more informations ;)

Copy link
Copy Markdown
Member

@emri99 emri99 left a comment

Choose a reason for hiding this comment

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

See comments on code ;)

Comment thread .gitignore Outdated
Comment thread package.json
Comment thread src/OauthReceiver/index.js
@emri99 emri99 changed the title Feature/post body compilation Fix: Resolve h is undefined error Feb 4, 2021
Comment thread .gitignore

# testing
/coverage
yarn.lock
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why ignoring this ?

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.

2 participants