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

Fix wanted: catleg article breadcrumbs #85

Closed
denismerigoux opened this issue Feb 16, 2024 · 2 comments · Fixed by #86
Closed

Fix wanted: catleg article breadcrumbs #85

denismerigoux opened this issue Feb 16, 2024 · 2 comments · Fixed by #86
Labels
enhancement New feature or request

Comments

@denismerigoux
Copy link
Contributor

Currently, catleg article LEGIARTI000044983201 output something like :

######## Article 13 | LEGIARTI000044983201

1. Le bénéfice ou revenu imposable est constitué par l'excédent du produit brut,
   y compris la valeur des profits et avantages en nature, sur les dépenses
   effectuées en vue de l'acquisition et de la conservation du revenu.

...

But actually what would be more useful is to have the headers in front, like :

# Code général des impôts

## Livre premier : Assiette et liquidation de l'impôt

...

####### Définition générale du revenu imposable

######## Article 13 | LEGIARTI000044983201

1. Le bénéfice ou revenu imposable est constitué par l'excédent du produit brut,
   y compris la valeur des profits et avantages en nature, sur les dépenses
   effectuées en vue de l'acquisition et de la conservation du revenu.
@denismerigoux denismerigoux added the enhancement New feature or request label Feb 16, 2024
@rprimet
Copy link
Contributor

rprimet commented Feb 16, 2024

OK, makes sense. What would be a good default behavior? Should we:

  • emit breadcrumbs by default, provide a flag to omit them
  • do not emit breadcrumbs by default, provide a flag to enable those
  • no flag, always print breadcrumbs?

@denismerigoux
Copy link
Contributor Author

emit breadcrumbs by default, provide a flag to omit them

I think this is best :)

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
None yet
Development

Successfully merging a pull request may close this issue.

2 participants