Skip to content
This repository has been archived by the owner on Jul 29, 2019. It is now read-only.

Fixed the two broken references to SweetAlert #3943

Merged
merged 2 commits into from
Jul 1, 2018

Conversation

sandebert
Copy link

The urls I used where the best ones I managed to find. (I'm sure there are better ones somewhere, but it was surprisingly difficult finding them.)

The urls I used where the best ones I managed to find. (I'm sure there are better ones somewhere, but it was surprisingly difficult finding them.)
@sandebert
Copy link
Author

Sorry, I should have mentioned this relates to http://visjs.org/examples/timeline/editing/editingItemsCallbacks.html, which is broken due to 404s for the SweetAlert references.

@Jogai
Copy link

Jogai commented Apr 23, 2018

Sweetalert introduced a lot of breaking changes, did you test with the 2.x releases? Mine broke with it, maybe use the links from https://www.jsdelivr.com/package/npm/sweetalert?version=1.1.3&path=dist

@sandebert
Copy link
Author

sandebert commented Apr 23, 2018

Oh, I made a bad assumption here. I just assumed the files had changed locations and that I just needed to update the references so I just updated the urls and PR'd.

So what do you say, are these reasonable urls instead?

@Jogai
Copy link

Jogai commented Apr 23, 2018

Yes, I would say so.

@yotamberk
Copy link
Contributor

hi @sandebert
I've now branched out to a new fork of my own including only Timeline.
I've applied your change here: https://github.com/yotamberk/timeline-plus
and can be used via npm here: https://www.npmjs.com/package/timeline-plus
Awesome fix! Thanks for your contribution! Future PRs will be reviewed and added in the new fork

@yotamberk yotamberk merged commit f5bf316 into almende:develop Jul 1, 2018
mojoaxel pushed a commit to visjs/vis_original that referenced this pull request Jun 9, 2019
* Fixed the two broken references to SweetAlert

The urls I used where the best ones I managed to find. (I'm sure there are better ones somewhere, but it was surprisingly difficult finding them.)

* Changed the urls again
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants