Skip to content

Edit analysis result#2551

Closed
tsundvoll wants to merge 1 commit intoequinor:mainfrom
tsundvoll:edit-analysis-result
Closed

Edit analysis result#2551
tsundvoll wants to merge 1 commit intoequinor:mainfrom
tsundvoll:edit-analysis-result

Conversation

@tsundvoll
Copy link
Copy Markdown
Contributor

@tsundvoll tsundvoll commented Jan 13, 2026

to match new format in SARA equinor/sara#243

Ready for review checklist:

  • A self-review has been performed
  • All commits run individually
  • Temporary changes have been removed, like console.log, TODO, etc.
  • The PR has been tested locally
  • A test have been written
    • This change doesn't need a new test
  • Relevant issues are linked
  • Remaining work is documented in issues
    • There is no remaining work from this PR that require new issues
  • The changes does not introduce dead code as unused imports, functions etc.

@github-actions
Copy link
Copy Markdown

🔔 Changes in database folder detected 🔔
Do these changes require adding new migrations? 🤔 In that case follow these steps.
If you are uncertain, ask a database admin on the team 😄

@tsundvoll tsundvoll force-pushed the edit-analysis-result branch from 80e9024 to cba3074 Compare January 13, 2026 19:42
to match new format in SARA equinor/sara#243
@tsundvoll tsundvoll force-pushed the edit-analysis-result branch from cba3074 to 0cc6ef7 Compare January 13, 2026 19:47
@github-actions
Copy link
Copy Markdown

🔔 Migrations changes detected 🔔
📣 Remember to comment "/UpdateDatabase" after review approval for migrations to take effect!

@github-actions github-actions bot added the database-change Will require migration label Jan 13, 2026
@tsundvoll
Copy link
Copy Markdown
Contributor Author

Contributes to #2513

@github-actions
Copy link
Copy Markdown

This pull request has been automatically marked as stale due to inactivity.

@github-actions github-actions bot added the stale This issue or pull request already exists label Feb 14, 2026
@andchiind andchiind closed this Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

database-change Will require migration stale This issue or pull request already exists

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants