Skip to content
This repository was archived by the owner on Mar 4, 2025. It is now read-only.

Sup 3018 webpack take two #698

Merged
merged 71 commits into from
Feb 2, 2016
Merged

Sup 3018 webpack take two #698

merged 71 commits into from
Feb 2, 2016

Conversation

nlitwin
Copy link
Contributor

@nlitwin nlitwin commented Feb 2, 2016

No description provided.

Nick Litwin and others added 30 commits January 24, 2016 21:59
-- Fixed member profile page to work according to the web pack restructuring.
-- Removed ngsticky because IMO, we are not using it anywhere. In case we need it again, I have it forked and published, as tc-ngsticky  npm package, with latest version for us.
-- >>Fork angular-intro and replace "intro" with "intro.js" on line 3 of angular-intro.js/src/angular-intro.js<< Done this but not sure why do we need it.
-- >>Use a different library for xml2js thats from NPM<< Used Leonidas-from-XIV/node-xml2js library to parse xml to JSON. IMO, it is used only in blog service.
-- >>Figure out why the error "Cannot read property 'split' of null" is happening from the angular-ellipsis library (account.title in linkedAccountsData null?)<< Fixed it. It was a bug in angular-ellipsis which is fixed in the original repo but the npm package still points to older version. Created new npm package, tc-angular-ellipsis , for pointing to latest version.
-- Fixed skills images to point to correct url
-- adding missing file from previous commit.
nlitwin added a commit that referenced this pull request Feb 2, 2016
@nlitwin nlitwin merged commit ad267b5 into dev Feb 2, 2016
@nlitwin nlitwin deleted the SUP-3018-webpack-take-two branch February 2, 2016 23:23
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.

1 participant