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 the dependency from Github #78

Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

Remove the dependency from Github #78

wants to merge 20 commits into from

Conversation

FilipZajdel
Copy link
Contributor

@FilipZajdel FilipZajdel commented Mar 21, 2025

Following changes are introduced:

  • Pull all information about addons and organization from the index instead from github.
    Add new fields in the schema to get the missing data that were previously fetched from github.

  • Add support button that displays e-mail and devzone username of the addon's contributor:
    image
    image

  • Update organizations' index files to align with introduced changes

  • Disabling telemetry when run in the local environment

  • Other minor changes such as dependencies update.

@FilipZajdel FilipZajdel changed the title Refactor the dependency on Github Remove the dependency from Github Mar 26, 2025
@FilipZajdel FilipZajdel requested a review from bencefr March 26, 2025 14:34
kylebonnici
kylebonnici previously approved these changes Mar 26, 2025
"title": "The Golioth Reference Design template",
"description": "This starter project is a standalone application that includes databases (both stateful and time-based), Over-the-air update capabilities, settings service, remote procedure calls (RPCs)",
"kind": "template",
"tags": ["dfu","lte"],
"avatar": "https://avatars.githubusercontent.com/u/58146410?s=48&v=4",
Copy link

Choose a reason for hiding this comment

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

This is independent issue that I noticed now, these avatar URLs should not contain the search parameters, they should be added (stripped and replaced) by the component rendering them.

VSC-2913

bencefr
bencefr previously approved these changes Mar 27, 2025
@FilipZajdel FilipZajdel dismissed stale reviews from bencefr and kylebonnici via e12c32b March 27, 2025 11:02
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.

3 participants