Sitelet https://github.com/Cumulocity-IoT/cumulocity-migration-tool/pull/6
Skip to content
This repository was archived by the owner on Sep 17, 2025. It is now read-only.

chore: optimized "getIdPathsFromDashboard" - #6

Open
simlu wants to merge 2 commits into
Cumulocity-IoT:masterfrom
simlu:patch-1
Open

simlu wants to merge 2 commits into
Cumulocity-IoT:masterfrom
simlu:patch-1

Conversation

@simlu

@simlu simlu commented Aug 19, 2019

Copy link
Copy Markdown

No description provided.

@simlu

simlu commented Aug 19, 2019

Copy link
Copy Markdown
Author

These should be equivalent. However significant faster (untested) and more readable :)

@rpeach-sag

Copy link
Copy Markdown
Contributor

Thanks for the pull request! I'll merge your changes when I've had a chance to test them. Probably not for a few weeks.

BTW your library is super handy!

@simlu

simlu commented Aug 21, 2019

Copy link
Copy Markdown
Author

No worries. I just couldn't resist opening the PR as it was such an obvious optimization

Cheers! Always great to see other people using the tools I built! It's now our new swiss army knife for data manipulation... 😄

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.

2 participants