Skip to content

feature/enable disable repositories#157

Merged
beslst merged 11 commits intomainfrom
feature/enable-disable-repositories
Dec 10, 2025
Merged

feature/enable disable repositories#157
beslst merged 11 commits intomainfrom
feature/enable-disable-repositories

Conversation

@beslst
Copy link
Copy Markdown
Contributor

@beslst beslst commented Nov 23, 2025

  • feat: support nuget repositories, which do not require a password
  • feat: allow to disable/enable repositories without removing them from the configuration
  • feat: store enabled flag correctly

@beslst beslst requested a review from seehma November 23, 2025 08:56
…on numbers - Beckkhoff compiles with four-part version numbers even they they use three-part version numbers in their nuspecs

(cherry picked from commit 560a143)
(cherry picked from commit 6b8d565)
@beslst
Copy link
Copy Markdown
Contributor Author

beslst commented Dec 4, 2025

@seehma friendly reminder :D

@seehma
Copy link
Copy Markdown
Contributor

seehma commented Dec 5, 2025

soll ich hier einfach mal alles a bissi durchtesten oder nur die neuen features?
meinst du mit store enabled flag correctly des
grafik

?

@seehma
Copy link
Copy Markdown
Contributor

seehma commented Dec 5, 2025

de meldung kommt und macht aber eigentlich keinen sinn oder? wenn ein repo disabled is dann braucht er auch nit prüfen ob die connection established werden kann.
grafik

Also was hab i gmacht, i hab bis auf den Twinpack server alle disabled, dann ok dann findet er keine beckhoff libs mehr
dann hab i des beckhoff repo wieder enabled, dann kommt de meldung, aber die connection zum beckhoff repo is ok, also er findet dann die libs -> deshalb geht i davon aus de meldung kommt wegen eines disabled repos in der liste der Connections

@seehma
Copy link
Copy Markdown
Contributor

seehma commented Dec 5, 2025

was ma grad auffallt aber nit zum thema passt -> cool wär vll. wenn man no sehen würde wann a library da auf twinpack online gangen is, nit? also Datum zumindest... Haben wir de info irgendwo?

@beslst
Copy link
Copy Markdown
Contributor Author

beslst commented Dec 5, 2025

Also was hab i gmacht, i hab bis auf den Twinpack server alle disabled, dann ok dann findet er keine beckhoff libs mehr
dann hab i des beckhoff repo wieder enabled, dann kommt de meldung, aber die connection zum beckhoff repo is ok, also er findet dann die libs -> deshalb geht i davon aus de meldung kommt wegen eines disabled repos in der liste der Connections

du bist nicht angemeldet bei beckhoff, deswegen die meldung. Die Meldung kommt für Enabled Repos, die keine geschlossene Chain haben ... Anmelden und sollte gehen

@seehma
Copy link
Copy Markdown
Contributor

seehma commented Dec 5, 2025

doch ich bin angemeldet bei beckhoff, sobald ich die meldung wegklicke geht alles ohne dass ich irgendwelche anmeldedaten eingeben muss. die anmeldedaten sind ja gespeichert von vorhin noch. kann des nit sein dass sich de meldung auf repos bezieht de nit angehakt sein?

@beslst
Copy link
Copy Markdown
Contributor Author

beslst commented Dec 5, 2025

man sieht in deinem screenshot, dass du nicht angemeldet bist. kannst bitte mal auf login drücken und machen, dass sich die kette schließt und schauen ob dann die meldung immer noch kommt?

@seehma
Copy link
Copy Markdown
Contributor

seehma commented Dec 5, 2025

ich hab auf dieses ja geklickt und die kette war geschlossen, sobald das abfragefenster weg war, da hab ich nicht auf login klicken müssen. siehe video im teams -> is ja nit dringend irgendwann halt...

@beslst
Copy link
Copy Markdown
Contributor Author

beslst commented Dec 5, 2025

Bug sollte behoben sein

@seehma
Copy link
Copy Markdown
Contributor

seehma commented Dec 10, 2025

ja schaut mal gut aus, jetzt kommt gar keine abfrage mehr, außer man ist wirklich ausgeloggt

Copy link
Copy Markdown
Contributor

@seehma seehma left a comment

Choose a reason for hiding this comment

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

Hab mir jetzt nicht den code durchgesehen, sondern einfach bei mir etwas rumgespielt. sollte passen

@beslst beslst merged commit a5fd8ad into main Dec 10, 2025
8 checks passed
@beslst beslst deleted the feature/enable-disable-repositories branch December 10, 2025 12:50
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.

2 participants