Little improvements #15
Conversation
|
|
||
| if len(project_candidates) == 1: | ||
| return project_candidates[0] | ||
| raise ValueError(f"project {project_name} not found or multiple projects with same name found") |
There was a problem hiding this comment.
Tohle bych rozdelil na dve chyby.
There was a problem hiding this comment.
No jasně ale tak jako ten kód na spoustu větších problémů tak zas další if se mi řešit nechtěl.
| @@ -669,7 +673,7 @@ def addEntry(self, start, description, projectName, userMail, workspace, | |||
| url = self.url + "/workspaces/%s/time-entries" % wsId | |||
|
|
|||
| if projectName != None: | |||
| projectId = self.getProjectID(projectName, workspace, skipPrjQuery=self._syncProjects) | |||
| projectId = self.get_project_id(projectName, workspace, skip_prj_query=self._syncProjects) | |||
There was a problem hiding this comment.
Proc neni _syncProjects take prejmenvoana?
There was a problem hiding this comment.
protoze je toho k prejmenovani milion :D asi bych
| @@ -795,7 +799,7 @@ def getTimeEntryForUser(self, userMail, workspace, description, | |||
|
|
|||
| def archiveProject(self, projectName, workspace, skipPrjQuery=False): | |||
There was a problem hiding this comment.
tady skipPrjQuery a projectName nejsou moc Python style
There was a problem hiding this comment.
Jako souhlas jen jsem začal pomalu přejmenovat a nechtělo se mi to dělat celý na to teď nemám čas
|
@marazt diky za review, ale fakt bych ted ty codestyles moc nehrotil a spis mrkni jen na tu logiku, to je jinak tak na 2 hodiny to vsechno prejmenovat a uklidit.. |
| billable = True | ||
| elif 'non-billable' in tags: | ||
| billable = False | ||
| else: |
There was a problem hiding this comment.
hele to bych nedelal, podle me kdyz tam neni explicitne billable tag, tak to chceme jako non-billable. Respektive, ja toggle takhle pouzivam...
There was a problem hiding this comment.
no takhle funguje clockify, ze kdyz je projekt jako celek neco tak podle toho se urci default.
ja takhle jsem zvyklej a dost mi to dava smysl, pro me je opruz pridavat furt billable vsude a prave mi vyhovuje ze vyuzivam toho, co je default na danym projektu.
ono je problem ze ty v podstate nikdy billable veci nemas, kdezto ja sem tam jo. Rekl bych ze aby obecne byl ten tool uzitecnej pro prumernyho mild blue cloveka, tak je moje reseni lepsi.
Ale pro tebe konkretne mozna chces mit ten case ze by default je vsechno non-billable i na billable projektech.
Tak tam asi muzeme pridat nejaky flag do configu no :D.
kubantjan
force-pushed
the
little-improvements
branch
from
154be9e to
f7712b4
Compare
…ased on project property. And couple of renamings.
kubantjan
force-pushed
the
little-improvements
branch
from
f7712b4 to
03c46b1
Compare
No description provided.