Skip to content

Conversation

@stefanvit
Copy link
Collaborator

Description

Add a POST/GET/PUT/DELETE API and meta_data collection in mongo
the object shoul look like this:
{
"id": "string",
"key": "string",
"value": {}, // map[string]interface{}
"date_created": "string",
"date_updated": "string",
}

**Resolves #125

Review Time Estimate

  • Immediately
  • Within a week
  • When possible

Type of changes

  • Bug fix (non-breaking change which fixes an issue).
  • New feature (non-breaking change which adds functionality).
  • Breaking change (fix or feature that would cause existing functionality to not work as expected).
  • Other (any another change that does not fall in one of the above categories.)

Checklist:

  • I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
  • I have updated the CHANGELOG.
  • I have read the Contributor Guidelines.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • My change requires updating the documentation.
  • I have made necessary changes to the documentation.
  • I have added tests related to my changes.
  • My changes generate no new warnings.
  • New and existing unit tests pass locally with my changes.
  • Any dependent changes have been merged and published in downstream modules.

@stefanvit stefanvit requested a review from mdryankov as a code owner October 7, 2025 11:47
@stefanvit stefanvit linked an issue Oct 7, 2025 that may be closed by this pull request
@stefanvit stefanvit requested a review from petyos October 7, 2025 11:47
# application
ContentItem:
$ref: "./application/ContentItem.yaml"
RewardClaim:
Copy link
Collaborator

Choose a reason for hiding this comment

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

What is this?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

To be tottaly honest i dont know from where this and ImageSpec came from for this issue, but i remove the from this PR

$ref: "./application/FileContentItemRef.yaml"
ImageSpec:
$ref: "./application/ImageSpec.yaml"
$ref: "./application/ImageSpec.yaml"
Copy link
Collaborator

Choose a reason for hiding this comment

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

What is this?

@stefanvit stefanvit requested a review from petyos October 8, 2025 10:36
petyos
petyos previously approved these changes Oct 8, 2025
Copy link
Collaborator

@petyos petyos left a comment

Choose a reason for hiding this comment

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

LGTM:
Fix docs.

type: string
url:
type: string
ImageSpec:
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should not be removed.

$ref: "./application/DataContentItem.yaml"
FileContentItemRef:
$ref: "./application/FileContentItemRef.yaml"
ImageSpec:
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should not be removed.

@stefanvit stefanvit merged commit 14b508e into develop Oct 8, 2025
2 checks passed
@stefanvit stefanvit deleted the 125-feature-meta_data-dependancies branch October 8, 2025 10:52
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.

[FEATURE] Meta_data dependancies

3 participants