Skip to content

fix: Fix sending metadata to API#97

Merged
skarahoda merged 1 commit intomainfrom
fix/metadata
Feb 1, 2022
Merged

fix: Fix sending metadata to API#97
skarahoda merged 1 commit intomainfrom
fix/metadata

Conversation

@skarahoda
Copy link
Copy Markdown

@skarahoda skarahoda commented Jan 31, 2022 •
edited
Loading

Change description

Type of change

  • Bug fix (fixes an issue)
  • New feature (adds functionality)

Related issues

Fix #1

Checklists

Development

  • Lint rules pass locally
  • Application changes have been tested thoroughly
  • Automated tests covering modified code pass

Security

  • Security impact of change has been considered
  • Code follows company security practices and guidelines

Code review

  • Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached as necessary
  • "Ready for review" label attached and reviewers assigned
  • Changes have been reviewed by at least one other contributor
  • Pull request linked to task tracker where applicable

@skarahoda skarahoda self-assigned this Jan 31, 2022
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jan 31, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@skarahoda skarahoda merged commit 6c7a438 into main Feb 1, 2022
@skarahoda skarahoda deleted the fix/metadata branch February 1, 2022 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@burakaraz burakaraz Awaiting requested review from burakaraz

@aykutbulca aykutbulca Awaiting requested review from aykutbulca

@yuksely yuksely Awaiting requested review from yuksely

@okan-cakmak okan-cakmak Awaiting requested review from okan-cakmak

1 more reviewer

@yuqu yuqu yuqu approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@skarahoda skarahoda

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@skarahoda @yuqu