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.

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

Remove @akveo/ng2-completer and ng2-completer #5896

Merged
merged 2 commits into from
May 20, 2022

Conversation

MaaxGr
Copy link
Contributor

@MaaxGr MaaxGr commented Oct 13, 2021

Please read and mark the following check list before creating a pull request (check one with "x"):

Short description of what this resolves:

Fixes #5814. smart-table dependency did change in version 1.7.0, see https://github.com/akveo/ng2-smart-table/blob/master/CHANGELOG.md

(I recreated the MR because i pushed the package-lock.json inadvertently)

@MaaxGr
Copy link
Contributor Author

MaaxGr commented Oct 13, 2021

Note: In Order to get rm package-lock.json && npm i --legacy-peer-deps && ng serve working again the following MRs must be merged:
#5894 #5896 #5898

(=> i let the package-lock.json untouched, so merging the MRs independently should not destroy anything and new users should be able to clone and start the project in every merge state. Only for web-developers who need a fresh package installation this will have an impact)

To see all changes of the three MRs together, see comparison in my fork: MaaxGr/ngx-admin@cabe4bd...910b4c4

ugurkoysuren
ugurkoysuren previously approved these changes May 12, 2022
@vaibhavkumar-sf
Copy link

It must be merged asap,
ref: #5893

elupanov
elupanov previously approved these changes May 20, 2022
@elupanov elupanov dismissed stale reviews from vaibhavkumar-sf, ugurkoysuren, and themself via 4a28d06 May 20, 2022 08:50
@elupanov elupanov merged commit 0f4156d into akveo:master May 20, 2022
@vaibhavkumar-sf
Copy link

  🤟

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.

Bug: Cannot start project
4 participants