search

EdisonReklamebyraa / woocommerce-bypost

0 stars 0 forks source link

Konfigurasjon lagres ikke #8

Closed espenlg closed 2 years ago

espenlg commented 2 years ago

Hvis man går inn til Bypost konfigurasjon ved å trykke på Rediger under Fraktsoner -> (Valgt sone) -> Bypost så lagres ikke det man endrer i Bypost konfigurasjon. Man må gå via WooCommerce -> Innstillinger -> Frakt -> Bypost for at endringer skal lagres.

Screenshot 2022-08-17 at 23 52 06

jarnalyrkar commented 2 years ago

Det virker som at de linker til to forskjellige steder, som er tilsynelatende like, men ikke helt:

/wp-admin/admin.php?page=wc-settings&tab=shipping&section=bypost_shipping_method (riktig) /wp-admin/admin.php?page=wc-settings&tab=shipping&instance_id=3 (feil)

har ikke løsningen på denne enda, men skal kikke videre

jarnalyrkar commented 2 years ago

Man kan i teorien legge til flere av samme fraktvalg, og hver av disse får hver sin instance_id:

image

Her er id'ene henholdsvis 3 og 10. Hvorfor ikke settings lagres på id-nivå vet jeg ikke enda.

jarnalyrkar commented 2 years ago

Fant greia her. Vi har nå slått på instance-settings til dette pluginet, men jeg mener at det blir feil. Man skal uansett bare legge inn priser, api-nøkler etc en gang, ett sted. Vi kan enkelt skru av instance-settings og da fjernes edit-knappen på shipping-sida, og man må inn på settingssida for å gjøre endringer.

Høres det OK ut?

jarnalyrkar commented 2 years ago

Instance settings er nå slått av, linje 31 i denne: https://github.com/EdisonReklamebyraa/bypost-woo/commit/02488fa14b2476454a06d041d729972378724586

Lukker.