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

feat: set hide_custom in user-modified Workspaces (LAN-45 ) #225

Closed
wants to merge 25 commits into from

Conversation

diePuppe
Copy link
Contributor

Hey Raffael,
wo kann ich die Workspace_hooks.py abspeichern, dass die nicht so wild dort herumliegt?

Grüße
Tim

@diePuppe diePuppe changed the title feat: LAN-45 hide custom allways 1 feat: hide custom allways 1(LAN-45 ) Jun 28, 2023
Copy link
Member

@barredterra barredterra left a comment

Choose a reason for hiding this comment

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

Ich würde die Datei in workspace.py umbenennen und dort liegen lassen, wo sie ist. Es gibt eigentlich einen Ordner, wo sie besser hinpassen würde.

Außerdem brauchen wir noch einen Patch, der diese Einstellung für die bestehenden angepassten Workspaces setzt (siehe Ordner und Datei patches) .

landa/workspace_hooks.py Outdated Show resolved Hide resolved
@barredterra barredterra changed the title feat: hide custom allways 1(LAN-45 ) feat: set hide_custom in user-modified Workspaces (LAN-45 ) Jun 29, 2023
diePuppe and others added 5 commits June 29, 2023 17:16
Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>
# Conflicts:
#	landa/workspace_hooks.py
landa/workspace.py Fixed Show fixed Hide fixed
@diePuppe
Copy link
Contributor Author

diePuppe commented Jun 29, 2023

Hey Raffael hab das mit dem pre-commit gemacht. Läuft auch durch alles durch, aber auf github kommen dann die ganzen Tests nicht durch.

grafik

@barredterra
Copy link
Member

barredterra commented Jun 30, 2023

@diePuppe sehr gut. Alles, was precommit betrifft,b läuft jetzt schonmal fehlerfrei durch.

Der eine Fehler war noch, dass du keine conventional commit Titel verwendet hast, aber das ist nicht so tragisch.

Und für den anderen Fehler kannst du nichts, den muss ERPNext beheben.

Aber die hooks.py sollten wir auf keinen Fall löschen!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants