Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Infocard for Entity Preview #2616

Merged
merged 25 commits into from
Aug 22, 2022
Merged

Infocard for Entity Preview #2616

merged 25 commits into from
Aug 22, 2022

Conversation

awildturtok
Copy link
Collaborator

No description provided.

@awildturtok awildturtok requested a review from thoniTUB July 26, 2022 10:28
Comment on lines +302 to +324
public static CQConcept forSelect(Select select) {
CQConcept cqConcept = new CQConcept();
cqConcept.setElements(List.of(select.getHolder().findConcept()));
CQTable table = new CQTable();
cqConcept.setTables(List.of(table));

table.setConnector(((Connector) select.getHolder()));

table.setSelects(List.of(select));

return cqConcept;
}

public static CQConcept forConnector(Connector source) {
final CQConcept cqConcept = new CQConcept();
cqConcept.setElements(List.of(source.getConcept()));
final CQTable cqTable = new CQTable();
cqTable.setConcept(cqConcept);
cqTable.setConnector(source);
cqConcept.setTables(List.of(cqTable));

return cqConcept;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Würde ich ein eine Util Klasse packen und doku bitte

@awildturtok awildturtok marked this pull request as ready for review August 17, 2022 13:04
@awildturtok awildturtok requested a review from thoniTUB August 17, 2022 13:04
@awildturtok awildturtok requested a review from thoniTUB August 22, 2022 08:04
thoniTUB
thoniTUB previously approved these changes Aug 22, 2022
Comment on lines +42 to +45
public boolean isSystem() {
// This Form should NEVER be started manually. Nor persisted
return true;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Der Kommentar klingt so als würde dem system Flag dieser Effekt anhaften, aber an sich ist es ein flag, dass eigentlich mal für das Frontend gedacht war.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ja ich hatte mir dann noch etwas Zeit genommen versuchen sie nicht zu persisten, dabei kam aber soviel blödsinn und verwirrung zusammen, dass ich es geculled habe :(

@@ -247,7 +247,7 @@ public void getConfigs() {
mode3.setForm(form);
mode3.setFeatures(List.of(new CQConcept()));

TestForm form3 = new TestForm();
TestFormRelUrl form3 = new TestFormRelUrl();
Copy link
Collaborator

Choose a reason for hiding this comment

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

hier werden wir gleich einen merge conflict haben

@awildturtok awildturtok requested a review from thoniTUB August 22, 2022 09:41
awildturtok and others added 3 commits August 22, 2022 12:01
# Conflicts:
#	backend/src/test/java/com/bakdata/conquery/api/form/config/FormConfigTest.java
#	backend/src/test/java/com/bakdata/conquery/api/form/config/TestFormAbsUrl.java
#	docs/Config JSON.md
Comment on lines +52 to +55
/**
* Used to map {@link SelectResultInfo} to {@link InfoCardSelect#getLabel()} via {@link PrintSettings#getColumnNamer()}.
*/
public String resolveSelectLabel(SelectResultInfo info) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Wo spielt der PrintSettings#getColumnNamer() hier rein?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Schau mal in com.bakdata.conquery.models.query.preview.EntityPreviewExecution#transformQueryResultToInfos

/**
* Find infoCard-selects by id within Dataset.
*/
public List<Select> resolveInfoCardSelects(Dataset dataset, DatasetRegistry registry) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

laut gh wird die methode in dieser Datei Zeile 395 benutzt 🤔

Copy link
Collaborator

Choose a reason for hiding this comment

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

Werden die Funktionen noch benutzt?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

com.bakdata.conquery.apiv1.QueryProcessor#getSingleEntityExport Zeile 395

@awildturtok awildturtok merged commit f9600ab into develop Aug 22, 2022
@delete-merged-branch delete-merged-branch bot deleted the feature/entity-extra-info branch August 22, 2022 15:06
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.

2 participants