Skip to content

feat(data_access): Add Required field to Endpoints table#1846

Merged
noaccOS merged 1 commit intoastarte-platform:masterfrom
nedimtokic:astarte-add-required-to-endpoints
Mar 11, 2026
Merged

feat(data_access): Add Required field to Endpoints table#1846
noaccOS merged 1 commit intoastarte-platform:masterfrom
nedimtokic:astarte-add-required-to-endpoints

Conversation

@nedimtokic
Copy link
Contributor

Add migration for new Required field in the Endpoints table, and use it.

@codecov
Copy link

codecov bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.90%. Comparing base (5f45ff1) to head (04f6d19).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1846      +/-   ##
==========================================
+ Coverage   85.85%   85.90%   +0.05%     
==========================================
  Files         560      560              
  Lines        9776     9776              
==========================================
+ Hits         8393     8398       +5     
+ Misses       1383     1378       -5     
Files with missing lines Coverage Δ
...keeping/lib/astarte_housekeeping/realms/queries.ex 91.54% <ø> (-0.36%) ⬇️
...te_data_access/lib/astarte_data_access/mappings.ex 100.00% <ø> (ø)
..._access/lib/astarte_data_access/realms/endpoint.ex 50.00% <ø> (ø)

... and 4 files with indirect coverage changes

Flag Coverage Δ
astarte_appengine_api 85.71% <ø> (+0.05%) ⬆️
astarte_data_access 82.55% <ø> (ø)
astarte_data_updater_plant 87.05% <ø> (+0.14%) ⬆️
astarte_events 79.40% <ø> (ø)
astarte_generators 97.74% <ø> (ø)
astarte_housekeeping 81.79% <ø> (-0.15%) ⬇️
astarte_pairing 84.65% <ø> (+0.11%) ⬆️
astarte_realm_management 87.66% <ø> (+0.07%) ⬆️
astarte_rpc 88.23% <ø> (ø)
astarte_trigger_engine 82.47% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nedimtokic nedimtokic force-pushed the astarte-add-required-to-endpoints branch 2 times, most recently from 8e1813a to cd617e6 Compare March 11, 2026 09:26
Copy link
Collaborator

@noaccOS noaccOS left a comment

Choose a reason for hiding this comment

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

🍜

@nedimtokic nedimtokic force-pushed the astarte-add-required-to-endpoints branch from cd617e6 to 555e431 Compare March 11, 2026 10:13
Add migration for new `Required` field in the Endpoints
table, and use it.

Signed-off-by: nedimtokic <nedim.tokic@secomind.com>
@nedimtokic nedimtokic force-pushed the astarte-add-required-to-endpoints branch from 555e431 to 04f6d19 Compare March 11, 2026 11:33
@nedimtokic nedimtokic requested a review from noaccOS March 11, 2026 11:43
@noaccOS noaccOS merged commit e1b066c into astarte-platform:master Mar 11, 2026
88 of 89 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants