entrouvert
/
combo
16
1
Fork 0
Code Issues Pull Requests 11 Releases Wiki Activity

dataviz: add an option to print values directly on chart (#68944) #253

Merged
yweber merged 1 commits from wip/68944-add-option-to-print-values-on-charts into main 2024-04-30 11:12:56 +02:00
Conversation 0 Commits 1 Files Changed 4 +52 -1
yweber commented 2024-04-22 14:30:34 +02:00
Owner
Copy Link
No description provided.
yweber added 1 commit 2024-04-22 14:30:34 +02:00
yweber changed title from WIP: dataviz: add an option to print values directly on chart (#68944) to dataviz: add an option to print values directly on chart (#68944) 2024-04-22 14:39:39 +02:00
vdeniaud requested changes 2024-04-22 15:22:02 +02:00
vdeniaud left a comment
Owner
Copy Link

Des broutilles, en te remerciant d'avoir pris ce ticket !

Des broutilles, en te remerciant d'avoir pris ce ticket !
combo/apps/dataviz/models.py Outdated
@ -291,0 +291,4 @@
print_values = models.BooleanField(
default=False,
verbose_name=_('Print values on chart'),
help_text=_('When not checked values are printed on hover'),
vdeniaud commented 2024-04-22 15:15:53 +02:00
Owner
Copy Link

Avec un point final pour être raccord avec les help_text du dessus (et en pensant à mettre à jour la migration :) )

Avec un point final pour être raccord avec les help_text du dessus (et en pensant à mettre à jour la migration :) )
yweber commented 2024-04-22 15:29:49 +02:00
Author
Owner
Copy Link

Bien vu, merci :)

Bien vu, merci :)
yweber marked this conversation as resolved
tests/test_dataviz.py Outdated
@ -1127,6 +1127,32 @@ def test_chartng_cell_hide_null_values_new_api(app, new_api_statistics):
]
@pytest.mark.parametrize(
vdeniaud commented 2024-04-22 15:18:00 +02:00
Owner
Copy Link

Ce bout me paraît inutile, voire sous-entend qu'on ferait quelque chose pour éviter d'appliquer cette option au type « Tableau » alors que non (ce qui pourrait faire l'objet d'un autre ticket, qui serait plus globalement faire du rangement dans ces options qui commencent à s'accumuler)

Bref, garder ce test très court en laissant le type de graphe par défaut serait mieux

Ce bout me paraît inutile, voire sous-entend qu'on ferait quelque chose pour éviter d'appliquer cette option au type « Tableau » alors que non (ce qui pourrait faire l'objet d'un autre ticket, qui serait plus globalement faire du rangement dans ces options qui commencent à s'accumuler) Bref, garder ce test très court en laissant le type de graphe par défaut serait mieux
yweber commented 2024-04-22 15:30:30 +02:00
Author
Owner
Copy Link

Bref, garder ce test très court en laissant le type de graphe par défaut serait mieux

Ca me va :)

> Bref, garder ce test très court en laissant le type de graphe par défaut serait mieux Ca me va :)
yweber marked this conversation as resolved
yweber force-pushed wip/68944-add-option-to-print-values-on-charts from 2026f4acaa to db5e3ded71 2024-04-22 15:29:34 +02:00 Compare
yweber requested review from vdeniaud 2024-04-22 15:32:43 +02:00
vdeniaud approved these changes 2024-04-22 15:47:18 +02:00
vdeniaud left a comment
Owner
Copy Link

Bon à merger au prochain cycle !

Bon à merger au prochain cycle !
combo/apps/dataviz/migrations/0030_chart_add_print_values_option.py Outdated
@ -0,0 +14,4 @@
name='print_values',
field=models.BooleanField(
default=False,
help_text='When not checked values are printed on hover',
vdeniaud commented 2024-04-22 15:47:02 +02:00
Owner
Copy Link

Manque le « . » ici :)

Manque le « . » ici :)
yweber commented 2024-04-22 15:52:23 +02:00
Author
Owner
Copy Link

Oh le piège ! Bien vu, merci :D

Oh le piège ! Bien vu, merci :D
yweber marked this conversation as resolved
yweber force-pushed wip/68944-add-option-to-print-values-on-charts from db5e3ded71 to dc124d92f1 2024-04-22 15:52:09 +02:00 Compare
yweber force-pushed wip/68944-add-option-to-print-values-on-charts from dc124d92f1 to f7e1b392af 2024-04-30 10:59:23 +02:00 Compare
yweber force-pushed wip/68944-add-option-to-print-values-on-charts from f7e1b392af to 04e1f69ff3 2024-04-30 11:09:15 +02:00 Compare
yweber merged commit 04e1f69ff3 into main 2024-04-30 11:12:56 +02:00
yweber deleted branch wip/68944-add-option-to-print-values-on-charts 2024-04-30 11:12:56 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
vdeniaud
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/combo#253
No description provided.
Delete Branch "wip/68944-add-option-to-print-values-on-charts"

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?