Skip to content

Add key based authorization to API Client#29

Merged
2 commits merged intov0.x.xfrom
unknown repository
Aug 14, 2024
Merged

Add key based authorization to API Client#29
2 commits merged intov0.x.xfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Aug 13, 2024

No description provided.

@ghost ghost requested a review from matthias-wende-frequenz August 13, 2024 11:36
@ghost ghost self-assigned this Aug 13, 2024
@ghost ghost self-requested a review as a code owner August 13, 2024 11:36
@github-actions github-actions Bot added the part:docs Affects the documentation label Aug 13, 2024
Copy link
Copy Markdown
Contributor

@daniel-zullo-frequenz daniel-zullo-frequenz left a comment

Choose a reason for hiding this comment

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

I only have an optional suggestion, LGTM otherwise

Comment thread src/frequenz/client/electricity_trading/_client.py Outdated
@ghost ghost force-pushed the add-authorization-key branch from 4024dba to deb1beb Compare August 14, 2024 09:49
camille-bouvy-frequenz added 2 commits August 14, 2024 12:23
Signed-off-by: camille-bouvy-frequenz <camille.bouvy@frequenz.com>
Signed-off-by: camille-bouvy-frequenz <camille.bouvy@frequenz.com>
@ghost ghost force-pushed the add-authorization-key branch from deb1beb to 9ab9491 Compare August 14, 2024 10:24
@ghost ghost added this pull request to the merge queue Aug 14, 2024
Merged via the queue into frequenz-floss:v0.x.x with commit 0c9dad9 Aug 14, 2024
@ghost ghost deleted the add-authorization-key branch August 14, 2024 10:43
@ghost ghost mentioned this pull request Aug 29, 2024
@ghost ghost added this to the v1.0.0 milestone Nov 14, 2024
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:docs Affects the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant