Skip to content

Conversation

@posva
Copy link
Contributor

@posva posva commented May 31, 2020

What kind of change does this PR introduce?

Typo in #4203

What is the current behavior?

What is the new behavior?

What steps did you take to test this? This is required before we can merge, make sure to test the flow you've updated.

  1. Step A
  2. Step B
  3. Step C

Checklist

  • Documentation
  • Testing
  • Ready to be merged
  • Added myself to contributors table

Well now I understand why it didn't work... I made a typo...

@lbogdan
Copy link
Contributor

lbogdan commented May 31, 2020

Build for latest commit 1663fdb is at https://pr4263.build.csb.dev/s/new.

@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 1663fdb:

Sandbox Source
Notifications Test Configuration

@CompuIves
Copy link
Member

Whoops 😂

@CompuIves CompuIves merged commit 0ed8fec into codesandbox:master May 31, 2020
CompuIves pushed a commit that referenced this pull request May 31, 2020
CompuIves added a commit that referenced this pull request Jun 1, 2020
* Added p5js icon

* Remove unnecessary fill property

* fix: setting undefined in URL (#4244)

* Remove pkg.json check (#4248)

* Remove pkg.json check

* Remove unused imports

* Add support for dynamic webpack loaders (#4245)

* Fix typo

* Undo the requires change

* Fix eslint (#4259)

* Resolve ESModules by default (#4191)

* Default to esmodules

* New packager

* Fix edge case with handling exports

* Test #1

* Test #2

* Test #2

* New try

* Patch resolve in babel

* Fix plugin

* Logging

* Remove esmodule check

* Change fallback

* Put check back

* Remove isEsModule check code

* Put convert esmodule code back in

* Optimize for speed

* Set a better default config

* Simplify boolean

* Set default babel configs

* Fix links

* Don't allow the sidebar to render if currentSandbox is not available (#4261)

* fix: check for vue router v4 (#4203)

* Fix esm var changes (#4262)

* Fix esm var changes

* Fix object initiliazers

* Add comment

* fix: typo in method isInsideVue (#4263)

* Fix transforming properties like toString

* Add comments data

* Use unique id

* Fix typo in reference

Co-authored-by: Ives van Hoorne <[email protected]>
Co-authored-by: Yash Joshi <[email protected]>
Co-authored-by: Ives van Hoorne <[email protected]>
Co-authored-by: Eduardo San Martin Morote <[email protected]>
CompuIves added a commit that referenced this pull request Jun 1, 2020
* Added p5js icon

* Remove unnecessary fill property

* fix: setting undefined in URL (#4244)

* Remove pkg.json check (#4248)

* Remove pkg.json check

* Remove unused imports

* Add support for dynamic webpack loaders (#4245)

* Fix typo

* Undo the requires change

* Fix eslint (#4259)

* Resolve ESModules by default (#4191)

* Default to esmodules

* New packager

* Fix edge case with handling exports

* Test #1

* Test #2

* Test #2

* New try

* Patch resolve in babel

* Fix plugin

* Logging

* Remove esmodule check

* Change fallback

* Put check back

* Remove isEsModule check code

* Put convert esmodule code back in

* Optimize for speed

* Set a better default config

* Simplify boolean

* Set default babel configs

* Fix links

* Don't allow the sidebar to render if currentSandbox is not available (#4261)

* fix: check for vue router v4 (#4203)

* Fix esm var changes (#4262)

* Fix esm var changes

* Fix object initiliazers

* Add comment

* fix: typo in method isInsideVue (#4263)

* Fix transforming properties like toString

* Add comments data

* Use unique id

* Fix typo in reference

* Don't show comment indication if you don't have permission

Co-authored-by: Danny Ruchtie <[email protected]>
Co-authored-by: Yash Joshi <[email protected]>
Co-authored-by: Eduardo San Martin Morote <[email protected]>
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.

3 participants