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.

Sign up for GitHub

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

Jump to bottom

Broken sourceMappingURL for dexie.min.js #326

Closed
adi130987 opened this issue Oct 5, 2016 · 5 comments · Fixed by #329 or #1676 · May be fixed by amacsnyk/Hygieia#4 or unnur28-snyktests/Hygieia#1
Closed

Broken sourceMappingURL for dexie.min.js #326

adi130987 opened this issue Oct 5, 2016 · 5 comments · Fixed by #329 or #1676 · May be fixed by amacsnyk/Hygieia#4 or unnur28-snyktests/Hygieia#1

Comments

@adi130987
Copy link

I just started using Dexie.js and I keep getting this console error:

Failed to parse SourceMap: http://localhost:8080/bower_components/dexie/dist/dist/dexie.min.js.map

I think dexie.min.js sourcemap url is broken:

//# sourceMappingURL=dist/dexie.min.js.map

it should be just:

//# sourceMappingURL=dexie.min.js.map
dfahlander added a commit that referenced this issue Oct 7, 2016
@dfahlander
Fixes #326 Broken sourceMappingURL for dexie.min.js.
f515dab 
uglify docs have no good answer on how to fix except by changing directory before uglifying. Fixed using a simple string.replace() instead. Also had to update uglify.
@dfahlander dfahlander mentioned this issue Oct 11, 2016
Merged
dfahlander added a commit that referenced this issue Oct 11, 2016
@dfahlander
Merge pull request #329 from dfahlander/develop
afbcd0c 
Bugfixes:
* Fixes #326 Broken sourceMappingURL for dexie.min.js

Build- and deployment tools
* Upgrading build and transpiler tools
* Prepared for supporting maintainance releases on 1.x branch while continuing to develop Dexie v2.x.
* Avoiding cyclic dependencies in ES6 source

Deprecations (See https://github.com/dfahlander/Dexie.js/wiki/Deprecations)
* Transaction.table
* Transaction.[table]
* Transaction.tables
* Transaction.error
* Transaction.complete
@zebzhao
Copy link

zebzhao commented Sep 21, 2017

Can you please reopen this? I downloaded the latest rc.1 release from bower and npm and it still points to dist/dexie.min.js.map

@dfahlander dfahlander reopened this Sep 21, 2017
@dfahlander
Copy link
Collaborator

Thanks. Regression bug.

dfahlander added a commit that referenced this issue Sep 22, 2017
@dfahlander
Resolves issue #326 again
f1b0a27 
This time using a simper approach.
This was referenced Mar 30, 2020
Open
Open
@lvivski
Copy link
Contributor

lvivski commented Feb 15, 2023

@dfahlander it looks like sourceMappingUrl is missing from the latest builds.

production: https://unpkg.com/dexie@3.2.3/dist/dexie.min.js
alpha: https://unpkg.com/dexie@4.0.1-alpha.7/dist/dexie.min.js

@dfahlander
Copy link
Collaborator

Thanks for reporting. It might be due to the upgraded dev-deps.

@lvivski
Copy link
Contributor

lvivski commented Feb 16, 2023

2.0.4 seems to be the last version to have sourceMappingUrl, starting with 3.0.0-alpha.1 it's gone

@lvivski lvivski mentioned this issue Feb 16, 2023
Merged
@lvivski lvivski mentioned this issue Feb 17, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@lvivski @adi130987 @zebzhao @dfahlander