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

Wer chunnt #27

Merged
merged 6 commits into from
Dec 18, 2019
Merged

Wer chunnt #27

merged 6 commits into from
Dec 18, 2019

Conversation

roranmorea
Copy link
Collaborator

No description provided.

@roranmorea roranmorea self-assigned this Dec 18, 2019
@roranmorea roranmorea added the enhancement New feature or request label Dec 18, 2019
@roranmorea roranmorea added this to 1. Checkpoint v1.0 in To do before release via automation Dec 18, 2019
@roranmorea roranmorea added this to the v1.0 milestone Dec 18, 2019
@roranmorea roranmorea moved this from 1. Checkpoint v1.0 to Check in To do before release Dec 18, 2019
_vorName,
_nachName,
_pos,
_homeFeedMainEventID,
Copy link
Owner

@pythonpoet pythonpoet Dec 18, 2019

Choose a reason for hiding this comment

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

Das sollte eine Liste sein

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Was sollte eine Liste sein?

Copy link
Owner

Choose a reason for hiding this comment

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

_homeFeedMainEventID oder kannst du mir erklähren für was das gebraucht wird?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

achso...
_homeFeedMainEventID war vorher _eventID, was auch ein String war keine Liste. Dieser String _eventID hatte keine Funktion und ihm wurde kein Wert übergeben. Ich habe diesen String dann umbenannt in _homeFeedMainEventID, dem der ID des ersten Events im homeFeed übergeben wird. Dadurch wird dann bei diesem Event eingelesen wer kommt und wer nicht.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

wenn du dies als Liste haben willst müssten wir einiges ändern gehen und ich denke dass sollte dann als neues Issue aufgemacht werden.

Copy link
Owner

Choose a reason for hiding this comment

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

oki

if (messagingGroups[u]) {
await crud0.setData('groups/$u/Devices', userUID, tokendata);
if (messagingGroups != null)
for (var u in messagingGroups.keys) {
Copy link
Owner

Choose a reason for hiding this comment

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

Ist jetzt messagingGroups im Midata group Format? also eine Nr. z.B. 4013 oder Drason (Buebe)? falls zweitens hier die Methode: convWebflowtoMiData(String stufe) einfügen

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

messaging Groups ist noch eine Map<String, bool> mit String = (z. B) Drason (Buebe)

Copy link
Owner

Choose a reason for hiding this comment

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

Ich meine wegen der Programmzeile unten
await crud0.setData('groups/$u/Devices', userUID, tokendata)
Wenn ich das richtig verstehe wird der Devicetoken mit Weblfow name gespeichert
grafik

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Also soweit ich weiss wird der Token unter der Midata Nummer gespeichert, weil als ich die MessagePage angepasst habe, ich die Senderliste fürs verschicken der Nachrichten aus dem Dokument mit der Midata Gruppennummer nehme und nicht aus dem Webflow Namen.
Aber sicher bin ich nicht. Auf jeden Fall habe ich nichts an dem geändert verglichen zum Masterbranch. In diesem Branch habe ich nur am WerChunnt gearbeitet.

info.getBemerkung(),
info.getSender(),
],
if (moreaFire.getPos != "Leiter") {
Copy link
Owner

Choose a reason for hiding this comment

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

Was ich vermeiden möchte ist ein Source-code der 700+ Zeilen hat. Ich finde dies unübersichtlich.
Aus diesem Grund habe ich angefangen das UI und die Logik zu trennen und zu seperieren. Vieleicht müssten wir und mal besprechen was in werches File kommt.
Mein Gedanke:
Die Klasse teleblitzAnmeldungen hat zwei Funktionen:
Stream<List<String>> getTNAngemolden(String eventID);
und
Stream<List<String>> getTNAbgemolden(String eventID);
Die übergeben dem Widget chunt und chuntnoed diesen Stream, der wiederum ein ListView generiert. Kannst du nicht einfach dies umschreiben und die Widgets aus Werchunt verwenden?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

aber genau das habe ich ja getan...
WerChunnt macht den stream und teleblitz zeigt ihn als listview an.

Copy link
Owner

Choose a reason for hiding this comment

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

Ja egal, wenn wir am gleichen arbeiten, müsste wir uns in Zukunft besser absprechen.

Copy link
Collaborator Author

@roranmorea roranmorea Dec 18, 2019

Choose a reason for hiding this comment

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

Ja das denke ich auch. Ich denke wir versuchen so schnell wie möglich eine stabile v.1 herauszubringen und dann können wir ja an der Übersichtlichkeit und Einheitlichkeit des Codes arbeiten.

Copy link
Owner

@pythonpoet pythonpoet left a comment

Choose a reason for hiding this comment

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

Habe ein Paar Komentare hinzu gefügt

@pythonpoet pythonpoet merged commit 76f3f6d into master Dec 18, 2019
To do before release automation moved this from Check to Done Dec 18, 2019
@roranmorea roranmorea deleted the wer-chunnt branch December 18, 2019 14:54
@roranmorea roranmorea mentioned this pull request Jan 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants