Skip to content
This repository has been archived by the owner on Nov 23, 2022. It is now read-only.

chore: data element insertion - lighthouse id - block listing #607

Merged
merged 13 commits into from
Nov 8, 2022

Conversation

Wagner3UB
Copy link
Collaborator

Us#34658

Nella sidebar dei blocchi elenco, è stato aggiunto un campo in cui si può scrivere scegliere un ID. Tale ID deve essere applicato dal personale RT come valore del data-element sul link delle card su blocchi elenco:

  • Card semplice
  • Card con immagine
  • in evidenza
  • Contenuto in evidenza
  • Card con nastro
  • Card con testo animato
  • Quadratoni con immagine
  • Blocco link completo

Il campo aggiunto è un select con:

  • service-category-link
  • topic-element
  • service-link
  • administration-element

@Wagner3UB Wagner3UB self-assigned this Nov 2, 2022
@github-actions
Copy link

github-actions bot commented Nov 2, 2022

Bundle size diff

Old size New size Diff
11.94 MB 11.94 MB -1.21 KB (-0.01%)

@SaraBianchi
Copy link
Contributor

Questa PR secondo me andava nei FEAT non nei CHORE

@pnicolli
Copy link
Member

pnicolli commented Nov 7, 2022

Questa PR secondo me andava nei FEAT non nei CHORE

Corretto. Quando la si mergia, fare "Squash and merge" e personalizzare il messaggio del commit risultante mettendo "feat: ". Se non è chiaro lo facciamo assieme.

Comment on lines 52 to 67
service_category_link: {
id: 'service-category-link',
defaultMessage: 'service-category-link',
},
topic_element: {
id: 'topic-element',
defaultMessage: 'topic-element',
},
service_link: {
id: 'service-link',
defaultMessage: 'service-link',
},
administration_element: {
id: 'administration-element',
defaultMessage: 'administration-element',
},
Copy link
Member

Choose a reason for hiding this comment

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

Penso che non serva che questi id siano traducibili in altre lingue, giusto? In tal caso semplificherei togliendo questi e togliendo sotto i vari intl.formatmessage() corrispondenti

src/config/Blocks/ListingOptions/utils.js Outdated Show resolved Hide resolved
@SaraBianchi SaraBianchi merged commit 0f03480 into master Nov 8, 2022
@SaraBianchi SaraBianchi deleted the us34658-data-element-block-listing branch November 8, 2022 09:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants