Skip to content

Krb-883 import old data#43

Draft
erik-itland wants to merge 11 commits into
mainfrom
KRB-0_import_old_data
Draft

Krb-883 import old data#43
erik-itland wants to merge 11 commits into
mainfrom
KRB-0_import_old_data

Conversation

@erik-itland
Copy link
Copy Markdown
Contributor

No description provided.

@erik-itland erik-itland force-pushed the KRB-0_import_old_data branch from 845176c to 77b4584 Compare January 5, 2023 04:50
@erik-itland erik-itland changed the title Krb 0 import old data Krb-883 import old data Jan 5, 2023
unique = true,
name = "ref"
)
@Column(unique = true)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Unikhet skal være innenfor prosjekt, ikke på tvers av prosjekter.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Men det her er vel ref-en til selve prosjektet. Om den ikke er unik på toppnivå så vil vi ikke kunne hente prosjekter basert på ref?

}
}

val publications = bank.publications.map { publication ->
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Denne gir ikke mye mening å importere

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Bare dersom en skal bruke det som testdata.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Feb 1, 2023

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 3 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

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.

3 participants