Skip to content

Install Select2 via npm#17023

Merged
skjnldsv merged 1 commit intomasterfrom
refactor/select2-from-npm
Sep 8, 2019
Merged

Install Select2 via npm#17023
skjnldsv merged 1 commit intomasterfrom
refactor/select2-from-npm

Conversation

@ChristophWurst
Copy link
Member

For #16956 (comment)

There was something about Select2 that made me install via git at #13156, but I can't remember what version at that time cause the problems.

@nextcloud/javascript where do we even use it? Couldn't find a single usage in server :)

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst ChristophWurst added 3. to review Waiting for reviews technical debt 🧱 🤔🚀 labels Sep 5, 2019
@ChristophWurst ChristophWurst added this to the Nextcloud 18 milestone Sep 5, 2019
@ChristophWurst ChristophWurst self-assigned this Sep 5, 2019
@skjnldsv
Copy link
Member

skjnldsv commented Sep 5, 2019

I think it is used by other apps ^^'
#1831

@ChristophWurst
Copy link
Member Author

Yeah. Would be pretty awesome if we tested at least with one app :)

Copy link
Member

@juliusknorr juliusknorr left a comment

Choose a reason for hiding this comment

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

Tagging and the workflow app still use it 😉 Just tested and it seems to be working as before.

@skjnldsv
Copy link
Member

skjnldsv commented Sep 8, 2019

Tagging and the workflow app still use it Just tested and it seems to be working as before.

I'll see what I can do about tagging for 18

@skjnldsv skjnldsv merged commit eafcb11 into master Sep 8, 2019
@skjnldsv skjnldsv deleted the refactor/select2-from-npm branch September 8, 2019 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants