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 C# Cloud Connector and Restructure cloud_connectors directory #58

Merged
merged 9 commits into from
Mar 25, 2024

Conversation

wilyle
Copy link
Contributor

@wilyle wilyle commented Mar 18, 2024

Removes the C# cloud connector in favor of the MQTT Cloud Connector. The C# Connector was an interim sample intended for a basic demo scenario, but the MQTT sample is now preferred for end-to-end sample scenarios. This PR consists of the following changes:

  • Remove the C# connector and all associated content, including the sample adapter that interfaces with it (fixes Remove digital twins cloud connector #44)
  • Consolidate content from the cloud_connectors/azure/README.md, cloud_connectors/azure/sample_dtdl/README.md, and cloud_connectors/azure/mqtt_connector/README.md file into the cloud_connectors/mqtt_connector/README.md file
  • Restructures the cloud_connectors directory so that it's organized more intuitively now that the above content is removed:
    • Removes the azure directory from the hierarchy since the connector is not Azure-specific
    • Moves the Azure function into a resources directory instead of being nested inside the MQTT connector source code
    • Moves all other content previously in the azure directory into the resources directory. This includes scripts and sample DTDL files.
    • Renames the sample-dtdl directory to sample_dtdl to be consistent about directory naming conventions

Copy link
Contributor

Alex Recommends Report

Alex recommends the following language changes, but Alex is a regular expression based algorithm, so take them with a grain of salt.

✨ 🚀 ✨ Nothing to Report ✨ 🚀 ✨

@wilyle wilyle changed the title Remove C# Cloud Connector Remove C# Cloud Connector and Restructure cloud_connectors directory Mar 18, 2024
@wilyle wilyle marked this pull request as draft March 18, 2024 22:41
@wilyle wilyle marked this pull request as ready for review March 25, 2024 16:40
freyja_apps/e2e/README.md Outdated Show resolved Hide resolved
freyja_apps/ibeji_integration/README.md Outdated Show resolved Hide resolved
freyja_apps/template/README.md Outdated Show resolved Hide resolved
Copy link

@ashbeitz ashbeitz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Several minor comments. Please take a look.

cloud_connectors/mqtt_connector/README.md Show resolved Hide resolved
cloud_connectors/mqtt_connector/README.md Outdated Show resolved Hide resolved
cloud_connectors/mqtt_connector/README.md Show resolved Hide resolved
cloud_connectors/resources/scripts/digital_twins_setup.sh Outdated Show resolved Hide resolved
cloud_connectors/resources/scripts/digital_twins_setup.sh Outdated Show resolved Hide resolved
freyja_apps/e2e/README.md Outdated Show resolved Hide resolved
Copy link

@ashbeitz ashbeitz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@wilyle wilyle merged commit f16863b into main Mar 25, 2024
14 checks passed
@wilyle wilyle deleted the wilyle/remove-cs-conn branch March 25, 2024 20:38
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.

Remove digital twins cloud connector
3 participants