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

[refactor]: migrations: use new "deep set" utility from npm to mutate input type to output type #59

Open
chrisguida opened this issue Jul 28, 2022 · 2 comments
Open

[refactor]: migrations: use new "deep set" utility from npm to mutate input type to output type #59

chrisguida opened this issue Jul 28, 2022 · 2 comments
Assignees
@Blu-J @chrisguida
Milestone
CLN v0.11.2.1

Comments

@chrisguida
Copy link
Contributor

This is useful for replacing the protectionless in-place substitution we're currently doing in the migration to convert booleans to objects and vice versa.

See: #54 (comment)
@chrisguida chrisguida added this to the CLN v0.11.2.1 milestone Jul 28, 2022
@chrisguida chrisguida moved this to Todo in Service Issues Jul 28, 2022
@chrisguida
Copy link
Contributor Author

@Blu-J can you link the issue in the sdk repo?

@chrisguida chrisguida mentioned this issue Jul 28, 2022
Closed
10 tasks
@chrisguida
Copy link
Contributor Author

https://deno.land/[email protected]/node/npm_specifiers
https://lodash.com/docs/4.17.15#set
import * as set from "npm:[email protected]/fp/set";

Screenshot 2023-02-23 at 16 37 21

@chrisguida chrisguida changed the title [refactor]: use new "deep set" utility once implemented in sdk [refactor]: migrations: use new "deep set" utility from npm to mutate input type to output type Feb 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Blu-J Blu-J

@chrisguida chrisguida

Labels
None yet
Projects
Service Issues
Status: Todo
Milestone
CLN v0.11.2.1
Development

No branches or pull requests
2 participants
@Blu-J @chrisguida