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

Adding new attributes to the HttpCustom model #295

Closed
wants to merge 2 commits into from

Conversation

RCX112
Copy link
Collaborator

@RCX112 RCX112 commented Jun 27, 2024

Extended the pydantic model with the attributes json, ngsi and timeout. Also added some tests for the pydantic model.

…). Also added some tests for the pydantic model.
@RCX112 RCX112 requested a review from djs0109 June 27, 2024 07:55
@djs0109
Copy link
Contributor

djs0109 commented Jul 3, 2024

@RCX112 I think there might be some misunderstanding. The related issue #290 should actually add headers under httpCustom. The related docs are here. I am really sorry for that. The implementation here is partially duplicated with #288, I am now looking into it and will see whether we can merge the two branch

@djs0109
Copy link
Contributor

djs0109 commented Jul 3, 2024

OK, I get it now. The headers already exists

@djs0109
Copy link
Contributor

djs0109 commented Jul 3, 2024

@RCX112 I will then merge #290 to #288 and then close the #290 . What do you think?

@RCX112
Copy link
Collaborator Author

RCX112 commented Jul 3, 2024

@djs0109 Sorry for the misunderstanding, we discussed this problem also in yesterday's meeting. I think merging both branches is a good idea

@djs0109
Copy link
Contributor

djs0109 commented Jul 3, 2024

Close, because it is merged to #288

@djs0109 djs0109 closed this Jul 3, 2024
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