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

Robustly handle config file syntax errors [PREVIEW] #19

Open
wants to merge 2,706 commits into
base: master
Choose a base branch
from

Conversation

nicoe
Copy link
Collaborator

@nicoe nicoe commented Aug 14, 2023

No description provided.

JCavallo added 30 commits June 14, 2023 17:06
electronic_signature: Add a timeout to signature service calls
…eout in conf returned by get_credentials (coopengo#23)

Fix #PROCK-283
…iled before attempting to reconcile it when posting a move [CUSTOM]

Fix #PJAZZ-703

When cancelling a move through the move cancellation wizard, we reconcile the lines and then post the generated cancellation move. However, when posting the move, we currently try to reconcile lines on which there is no credit/debit, which results in an error because these lines have already been reconciled in the previous step.
fixed bugs and implemented coog specific features

Fix #3643
http://rietveld.coopengo.com/20970001/
dbclick on menu
dbclick on treeviews
hidden checkboxes in trees
fix #4830
call group.remove() only when necessary
compare for undefined when compared to null
fix #4752
JCavallo and others added 24 commits July 3, 2023 15:56
* Add missing part of ab3dd96 [CUSTOM]

* Return a compatible type for multi valued fields in unique_value [PREVIEW]

* Restore lock_signal functionality on group [CUSTOM]

* Fix missing part from 5.2 merge [CUSTOM]
Comment on lines 514 to 517
logger.error(
f"Failed to parse {self.profiles_cfg}. "
f"A backup can be found at {temp_name}",
exc_info=True)
Copy link
Contributor

Choose a reason for hiding this comment

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

Au delà du log, ça ne pourrait pas être un message utilisateur ? Sous windows, ils sont à peu près 0% à regarder les logs je pense :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oui on pourrait :), je pense qu'on a un utilitaire qui devrait le permettre.

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.

3 participants