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

Conversation

@codydjango
Copy link

Ticket

PLAT-XXXX
Related tickets:
Related PRs:

Type of PR

  • Bugfix
  • New feature
  • Minor changes

Did you make changes to modules or created a new module?

Changes introduced

Describe the changes being introduced in this PR
Include screenshots if necessary
Please link any documentation reference that relate to the changes

Test and review

Describe how a reviewer should test your PR

robester0403 and others added 10 commits February 15, 2024 11:53
…cript (#1096)

* PLAT-13621 changed command name from crowdbotics-module to cb

* PLAT-13621 changed command in console log and comments to cb

* PLAT-13621 updated readme

* PLAT-13621 initially added pre execute checks

* PLAT-13621 Refactored out the Django check so we can run more specific checks when Django is absolutely needed

* PLAT-13621 added back amplitude

* PLAT-13621 Added more checks

* PLAT-13621 changed check to reading rc files.

* PLAT-13621 Fixed version logic

* PLAT-13621 Removed checks for demo

* PLAT-13621 changed some invalid back to section

* PLAT-13621 Removed unneeded node checks

* PLAT-13621 Added regex check for python

* PLAT-13621 Removed unused import

* PLAT-13621 reverted

* PLAT-13621 removed unused check

* PLAT-13621 added check to demo without cookiecutter since it attempts to install it in the script

---------

Co-authored-by: Shashank Sinha <shashank@crowdbotics.com>
Co-authored-by: Dan DeFelippi <dan@driverdan.com>
Co-authored-by: Crowdbotics <team@crowdbotics.com>
* PLAT-13819: Replace Amplitude with Segment

* Update: replace key values

* Remove unused async

* update: module tracking enhancements as per PM requests

* update: track anonymous users; track other commands (per request)

* fix failed CI; remove magic strings

* fix: minor update
hotfix: properly check for correct npx --yes check
@codydjango codydjango requested a review from driverdan March 12, 2024 03:40
@driverdan driverdan merged commit 079cc8e into master Mar 12, 2024
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.

8 participants