search

inventaire / inventaire-client

webapp coupled to the inventaire server :books:
https://inventaire.io
49 stars 16 forks source link

Add entity links to edition layout #393

Closed jum-s closed 1 year ago

jum-s commented 1 year ago

Address #391 as a prefiltering function was removing too many claims from edition layout, preventing to display bibliographical databases links (solved by commit 58d0ab281ae9809d1fd6727f122cec5ef4b30afc) This PR also take advantage of context to push some cosmetic commits

jum-s commented 1 year ago

Which entity links is it adding?

That is the issue linked above (#391): ie. BNF id wasn't displayed. It should be with 58d0ab2

I wish we were going more in the direction of clearly showing what belongs to the work and what belongs to the edition

Agree on the theory, difficult in practice. It make sense to me that an author is the second thing you see on an edition page after the edition's title

I guess it's ok to display more work properties

That is not the case, the idea was feature parity with backbone layout, hence only a handful of work props are displayed (cf. comment above and editionWorkProperties).