feature/pricelist add server pagination for balance #17
No reviewers
Labels
No Label
🛃 types
👪 users
invalid
wontfix
🚀 announcement
🎒 backend
🐞 bug
🐋 docker
💡 enhancement
📺 frontend
🤔 question
🔗duplicate
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: Flaschengeist/flaschengeist#17
Loading…
Reference in New Issue
No description provided.
Delete Branch "feature/pricelist"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
first pull request was wrang branch
Hab mal bissl kommentiert, generell das mit dem virtuellen User ist mal mega nice!
s. Kommentare
@ -120,3 +120,2 @@def get_users():return User.query.all()def get_users(userids=None):Siehe unten: https://flaschengeist.dev/Flaschengeist/flaschengeist/pulls/17/files#issuecomment-164
Bez. die funktion im controller ist schon ok, aber damit wird alles immer sortiert, nicht so sicher ob das sortieren hier immer sinnvoll ist.
Also a] ob es sinnvoll ist das immer zu tun und b] ob die sortierung hardcoded Sinn ergibt (z.b. meist sieht man ja die User mit dem display_name und da kann das z.b. manchmal zu komischen ergebnisse führen)
Ich dachte, ich arbeite schon mal vor, falls wir hier auch server side pagination brauchen. demzufolge, können wir das hier auch wieder raus nehmen.
@ -195,6 +199,8 @@ def register(data):)messageController.send_message(messageController.Message(user, text, subject))find_user(user.userid)Sieht für mich ziemlich unnötig aus, gibt es einen Grund das hier aufzurufen?
Damit werden gleich Userattributes geupdatet. Ohne diese Funktion, gibt es keine "DN" und es erschien mir einfacher, das einfach damit aufzurufen. (Benutze ich auch in run_flaschengeist ldap_sync)
Allgemein, sollte das eigentlich in einem anderen commit rein.
@ -104,6 +104,11 @@ class User(db.Model, ModelSerializeMixin):def has_permission(self, permission):return permission in self.get_permissions()def __repr__(self):Sieht nach debugging aus? Ist das für logs gedacht? Dann vielleicht lieber
__str__als__repr__(repr sollte eindeutig sein, das string ist aber nicht per-se eindeutig).Ja dies war für debugging. Evtl. könnten wir sowas ja auch Einführen dass wir
__str__und__repr__für models einführen. (Liest sich schöner im Log)@ -43,3 +42,1 @@credit = db.session.query(Transaction.receiver_id, func.sum(Transaction.amount)).filter(Transaction.receiver_ != None)def get_balances(start: datetime = None, end: datetime = None, limit=None, offset=None, descending=None, sortBy=None):Der Ansatz ist cool! NICE 👍
Noch irgendwas hier zu machen?
@ -113,2 +113,2 @@users = userController.get_users()userids = Noneif "userids" in request.args:Siehe unten: https://flaschengeist.dev/Flaschengeist/flaschengeist/pulls/17/files#issuecomment-164
@ -70,3 +70,3 @@"""logger.debug("Retrieve list of all users")users = userController.get_users()userids = NoneWird die Änderung eigentlich irgendwo verwendet?
Die widerspricht dem bisherigen REST Ansatz, daher entweder einzelner Datensatz (
/user/xy) oder einem Block (/usersbez./users?limit...&offset...).Ich kann mir denken wofür die gedacht ist, aber ich bin mir nicht so sicher ob das sinnvoll ist.
Wird im Plugin verwendet. Damit braucht man nicht mehr alle Users laden (was im übrigen irgendwann ja sehr viele sein werden) sondern nur ein Bruchteil direkt über die userids.
guckst du hier
11b7f05ad7toc3468eea03