Giter VIP home page Giter VIP logo

delivery-options's People

Contributors

alexmaat avatar dependabot[bot] avatar edielemoine avatar firstred avatar gravendeeljochem avatar joerivanveen avatar mark-ernst avatar myparcel-bot[bot] avatar richardperdaan avatar semantic-release-bot avatar wthijmen avatar

Stargazers

 avatar

Watchers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar

Forkers

robbinbenard

delivery-options's Issues

Zondag afleverdag voor maandag levering

Wanneer je op dit moment zondag als afleverdag instelt wordt er niets mee gedaan. kan dit ervoor zorgen dat in de checkout maandag als leverdag te zien is?

Create option to show list or map or both

Using plugin 4.1.4 on Magento 2.3.6

I would like the plugin to make it optional to show the map for pickup points. Reason for this is that the Belgium pickup points are not showing on the map, it only confuses customers. So a simple option to show only list, map, or both would suffice.

Add Carrier - DHL Europlus

What problem does this feature solve?

We need this feature to show this newly implemented carrier from the API.

What should the solution look like?

Add the Carrier DHL Parcel Connect with carrier_id 11 to the CarrierConfiguration

Relevant log output

No response

Additional context

No response

Show base price of delivery/pickup option

Describe the solution you'd like
The customer should see the base price of either option before selecting them.

For example: If there is a discount for using pickup locations customers don't know this until they actually select a pickup location. If they could see there is a discount they will be more tempted to select pickup instead of delivery.

Additional context
Example of what it could look like:
Screenshot 2020-09-16 at 17 50 48

Leaflet error on incorrect address

Describe the bug
Leaflet throws an error and breaks when the centerpoint address is incorrect.

myparcel.js?ver=4.1.4:7 Error: Bounds are not valid.
    at i.fitBounds (leaflet.js:formatted:1436)
    at a.fitToMarkers (myparcel.js?ver=4.1.4:7)
    at a.<anonymous> (myparcel.js?ver=4.1.4:7)
    at Array.<anonymous> (myparcel.js?ver=4.1.4:7)
    at ut (myparcel.js?ver=4.1.4:7)

Configuration

{
  "config": {
    "platform": "belgie",
    "locale": "nl-BE",
    "currency": "EUR",
    "carrierSettings": {
      "bpost": {
        "allowDeliveryOptions": true,
        "allowPickupLocations": true,
        "largeFormat": false,
        "allowSignature": false,
        "allowOnlyRecipient": false,
        "cutoffTime": "14:00",
        "deliveryDaysWindow": 1,
        "dropOffDays": [
          "1",
          "2",
          "3",
          "4",
          "5",
          "6"
        ],
        "dropOffDelay": 1,
        "pricePickup": -2,
        "priceSignature": 1,
        "priceOnlyRecipient": 0,
        "priceSaturdayDelivery": 0
      },
      "dpd": {
        "allowDeliveryOptions": true,
        "allowPickupLocations": true,
        "largeFormat": false,
        "allowSignature": false,
        "allowOnlyRecipient": false,
        "cutoffTime": "17:00",
        "deliveryDaysWindow": 1,
        "dropOffDays": [
          "1",
          "2",
          "3",
          "4",
          "5"
        ],
        "dropOffDelay": 0,
        "pricePickup": -2.5,
        "priceSignature": 0,
        "priceOnlyRecipient": 0,
        "priceSaturdayDelivery": 0
      },
      "postnl": {
        "allowDeliveryOptions": true,
        "allowPickupLocations": true,
        "largeFormat": false,
        "allowSignature": true,
        "allowOnlyRecipient": false,
        "cutoffTime": "17:00",
        "deliveryDaysWindow": 1,
        "dropOffDays": [
          "1",
          "2",
          "3",
          "4",
          "5"
        ],
        "dropOffDelay": 0,
        "pricePickup": -2,
        "priceSignature": 0,
        "priceOnlyRecipient": 0,
        "priceSaturdayDelivery": 0
      }
    }
  },
  "strings": {
    "addressNotFound": "Adresgegevens zijn niet ingevuld",
    "city": "Plaats",
    "closed": "Gesloten",
    "deliveryStandardTitle": "Standaard levering",
    "deliveryTitle": "Thuis of op het werk bezorgd",
    "headerDeliveryOptions": "Bezorgopties",
    "houseNumber": "Huisnummer",
    "openingHours": "Openingstijden",
    "pickUpFrom": "Ophalen vanaf",
    "pickupTitle": "Afhalen ophaalpunt",
    "postcode": "Postcode",
    "retry": "Opnieuw",
    "wrongHouseNumberCity": "Postcode/plaats combinatie onbekend",
    "signatureTitle": "Handtekening voor ontvangst ",
    "onlyRecipientTitle": "Alleen ontvanger",
    "pickupLocationsListButton": "Lijstweergave",
    "pickupLocationsMapButton": "Kaartweergave"
  },
  "address": {
    "cc": "de",
    "postalCode": "200",
    "number": "200",
    "city": "Hamburg"
  }
}

Expected behavior
Map doesn't show and asks to correct the address.

Screenshots
Screenshot 2020-09-29 at 12 15 53

Hardware

  • OS: MacOS 10.15.4
  • Browser: Chrome 85.0.4183.121

Selected location of PostNL locations in list not clearly recognizable for customer

Describe the bug
When I try to select a location of my preference in the checkout using "List" tab (new design in version 4.x.x) it is not clear to me 1) if I selected a location and 2) what location. The background is white, there is no difference to see between the selected location and the other alternatives. This is not clear to the customer and will lead to confusion

To Reproduce
Steps to reproduce the behavior:

  1. Go to checkout
  2. shipping section
  3. select the option for
  4. use the list view and try to select a location of your preference

Expected behavior
Design that shows a clear difference in f.i. color or a text that says selected or something similar.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: iOS mac
  • Browser Chrome and Safari
  • MyParcel Plugin version 4.1.1
  • MyParcel SDK version 5.1.0
    also using latest Mageplaza OSC 2.9.5
    Magento version 2.3.5-p1

Additional context
Add any other context about the problem here.

As a client I also want to show the street of the pick up point

Is your feature request related to a problem? Please describe.
There are multiple pickup points with the same name in my area which will be displayed at the top of the pick up list.

Describe the solution you'd like
Showing the street of the pickup point

Add Carrier - DHL Parcel Connect

What problem does this feature solve?

We need this feature to show this newly implemented carrier from the API.

What should the solution look like?

Add the Carrier DHL Parcel Connect with carrier_id 10 to the CarrierConfiguration.

Relevant log output

No response

Additional context

No response

Standard delivery not in the right position

Describe the bug
Standard delivery sometimes shows AFTER morning and evening delivery

Configuration

{
  "config": {
    "carriers": [],
    "platform": "myparcel",
    "locale": "nl-NL",
    "currency": "EUR"
  },
  "strings": {
    "addressNotFound": "Address details are not entered",
    "city": "City",
    "closed": "Closed",
    "deliveryStandardTitle": "Standard delivery",
    "deliveryTitle": "Delivered at home or at work",
    "deliveryMorningTitle": "Morning delivery",
    "deliveryEveningTitle": "Evening delivery",
    "headerDeliveryOptions": "",
    "houseNumber": "House number",
    "openingHours": "Opening hours",
    "pickUpFrom": "Pick up from",
    "pickupTitle": "Pickup",
    "postcode": "Postcode",
    "retry": "Retry",
    "wrongHouseNumberCity": "Postcode/city combination unknown",
    "signatureTitle": "Signature on delivery",
    "onlyRecipientTitle": "Home address only"
  },
  "address": {
    "cc": "nl",
    "postalCode": "1025 WK",
    "number": "576",
    "city": "Amsterdam"
  }
}

Expected behavior
Standard delivery is between morning and evening delivery.

Screenshots
Screenshot 2020-09-16 at 17 37 23

Hardware

  • OS: MacOS 10.15.4
  • Browser Chrome 85.0.4183.102

Allow pre-defined delivery options

Is your feature request related to a problem? Please describe.
If the customer decides to adjust his delivery options after the data was sent and he left the page, going back will require him to fill in the information from nothing. I could not find a option to pre-fill the delivery options on render or update events.

Describe the solution you'd like
Add "delivery_options" to window.MyParcelConfig with the structure of the contents of #mypa-input that will allow customers to edit what they picked previously.

If I missed some configuration that can make this happen, please let me know. Thank you.

Improve experience when there are no options available

Is your feature request related to a problem? Please describe.
When there are no pickup locations it doesn't handle this correctly. It just errors and hides while at home delivery may still be possible.

Describe the solution you'd like
When this occurs, keep showing the module but show a message along the lines of "No pickup locations found."

Additional context
Relates to myparcelnl/woocommerce#377

Validate configuration

Is your feature request related to a problem? Please describe.
You're able to enter anything in configuration properties, this can easily cause unexpected behaviour which can be hard to debug.

Describe the solution you'd like
Validate the configuration on load and report warnings if types are invalid, unsupported options are passed, etc.

Incorrect instructions on non existing address for pick up location finder

When using a faulty address on purpose. And selecting 'pick up from location' the error message pop's up that their has no address has been filled in.

There has. But the supplied address doesn't exist. For more info see attached image.

Additional context
Add any other context about the problem here.
woocom-myparcel

Date selector pick list in checkout is displaying "Invalid day" as title for each day to select

Describe the bug
Date selector pick list in checkout is displaying "Invalid date" as title for each day to select

To Reproduce
Steps to reproduce the behavior:

  1. Put products in basket
  2. Go to checkout
  3. Go to shipping section
  4. When trying to select a delivery day, the pick list shows "invalid date" for all values
    Strangely it only happens with iOS/Mac devices

Expected behavior
Show dates instead of "invalid date" text

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: iOS and Mac
  • Browser: Safari
  • MyParcel Plugin version: 4.1.1
  • MyParcel SDK version: 5.1.0

Make more options for map/list view when choosing a pick-up location

Please make the list/map view when choosing a pickup location more configurable. Let the shopowner choose what to show and what should have priority.

Reasons:

  • Not every store wants to show the map
  • The map doesn't work correctly (bug filed)
  • As a shopowner I want to be able to choose the listview as default; there the shipment costs are displayed rightaway, not after you've clicked a location

Stickymenu issue

Using MyParcel 4.1.6-beta.1 and Magento 2.3.5 together with Firecheckout.

We have a sticky horizontal menu in the header.
When we select an address the deliveryoptions show up.
Select pickup location, witness the pickup list expanding, select more options.
Now scroll down to the bottom of the site and witness the stickymenu going awry.

Reason for this is that the angular box isn't adding up to the document size. I tried to observe it with ResizeObserver, but that doesn't play nice with Angular.

I now circumvented it in a different way, but it would be nice for the document to know how high the deliveryoptions container is so that sticky elements remain working.

Verschillende sluitingstijden

Het is mogelijk dat sommige dagen een andere sluitingstijd hebben. Kan het zo gemaakt worden dat je per dag kan instellen wanneer je sluit, zodat de checkout hier goed op ingesteld kan worden?

Afhaalpunten België en Nederland apart aan of uit zetten

Graag de mogelijkheid om de mogelijkheid voor het kiezen van afhaalpunten voor België en Nederland apart aan of uit te zetten. Dit was tijdens het begin van de Corona-perikelen een probleem, omdat België al de pakketpunten had gesloten en Nederland niet.

Feature request: Aanpassen volgorde bezorgopties

Als een klant wil ik de volgorde van de verzendmethoden in de checkout (standaard / ochtend / avond) willen aanpassen.
Nu is de volgorde:

  • Ochtend
  • Avond
  • Standaard

Als ik dit zelf kan wijzigen, kan iedereen zelf bepalen wat het best voor hem/haar werkt.

Changing the configuration with "myparcel_update_config" doesn't work

It's currently not possible to use the event myparcel_update_config to change the carriers on the fly. You'll end up only being able to select the first available carrier. It's possible that this event triggers more bugs, if you run into any please report them here so we can test these as well.

Workaround

A workaround for now is to do a rerender, by essentially deleting the whole html structure of the delivery options, placing a new empty div with the id myparcel-delivery-options, and sending an event to myparcel_render_delivery_options.

Example

const oldElement = document.querySelector('.myparcel-delivery-options');
oldElement.outerHTML = '<div id="myparcel-delivery-options"></div>';document.dispatchEvent(new Event('myparcel_render_delivery_options'));

Note

By @gett-thijssimonis

The event myparcel_update_config also requires that config param currency is present instead of using default from defaultConfiguration.

Error

TypeError: Currency code is required with currency style.

As a client I want the map to zoom out until PostNL pickup points are visible

Describe the bug
When I use an address with no pickup points nearby the map shows nothing but a blank map. Because of this you have to zoom out until there are any pickup points to be showed.

Steps to reproduce

  1. Go to 'the checkout'
  2. Use postal code 9982BL with house number 2
  3. Click on 'pick up'
  4. See no pickup points on the map

Expected behavior
The map should zoom out until a few pickup points are visible.

Add the address of the location to the list view

Currently the list view only mentions the location name but not the streetname. Especially when there are multiple retail locations with the same brand name mentioned in the list, this becomes confusing.

Solution: Add a streetnaam+housenumber below the retail location name in the list view.

Default pickup location on map stays highlighted when selecting another location

Describe the bug
When selecting a new location, the default location will not appear "unselected". This way the user can't see which one is the "real" selected location.

Configuration
Default configuration on https://myparcelnl.github.io/delivery-options/

Steps to reproduce

  1. Go to pickup map
  2. Click on an unselected location
  3. Original location is not deselected.

Expected behavior
Original location is deselected.

Screenshots
Screenshot 2020-10-02 at 11 11 11

Hardware

  • OS: MacOS 10.15.4
  • Browser: Chrome 85.0.4183.121

Pickup Locations not visible in Belgium.

With a clean install. With all the latest versions of WP, WOO and Myparcel (on 7-12-20) it seems that pickup locations are excluded or not loaded on the map for Belgium.
When moving in the map the icon of the locations are visible, but only in The Netherlands.

The list without map works.

OnPaste 20201207-194314
OnPaste 20201207-194242

Cut-off tijd per dag instellen

Graag zou ik de sluitingstijden per dag in willen stellen. Ik heb nu doordeweeks een cut-off tijd van 23:00 uur en op Zondag een cut-off-tijd van 18:00 uur.
Daarnaast als ik dan een cut-off tijd op Zondag instel moet de afleverdag op Maandag worden vermeld en niet op Dinsdag zoals nu het geval is.)
(Alle bestellingen van vrijdag 23:00 uur tot zondag 18:00 uur worden op maandag bezorgd...)

Radio button disabled when only a single option is available.

Describe the bug
The delivery options radio button is disabled, but selected, when only one MyParcel option is set. If the user has any other delivery methods besides MyParcel, MyParcel options will always be selected.

Expected behavior
Unselect MyParcel shipping method when another delivery method is available.

Screenshots
image

Allow use of street instead of number

Describe the solution you'd like
The delivery options endpoint now supports sending street instead of number. This should be supported so frontends don't have to split numbers from full street addresses if they don't use separate address fields.

Show more specific error messages

Is your feature request related to a problem? Please describe.
Right now, only a generic message will be shown when the API returns an error.
Describe the solution you'd like
It would be good to show what exactly went wrong, for example when an invalid address was entered.
Describe alternatives you've considered
.
Additional context
Social Shipping 2021-07-06 14-00-42

Improve package type handling

Describe the bug
When package type is not "package", pickup locations aren't automatically turned off, forcing anyone implementing the delivery options to write this logic themselves.

Configuration

{
  "config": {
    "packageType": "digital_stamp",
    "platform": "myparcel",
    "carrierSettings": {
      "postnl": {
        "allowStandardDelivery": true,
        "allowPickupLocations": true,
        "allowEveningDelivery": false,
        "allowMondayDelivery": false,
        "allowMorningDelivery": false,
        "allowOnlyRecipient": false,
        "allowSaturdayDelivery": false,
        "allowSignature": false
      }
    }
  }
}

Steps to reproduce

  1. Load delivery options with above config
  2. See pickup option

Expected behavior
No pickup option to be available.

Centerposition of map doesn't match entered location

In version 4.1.1 the shown map has a green dot in the center, most likely based on the entered location (I'd expect). However, it doesn't match that location (checked with 5 postcodes/addresses) - in one case I came in a totally different village.

However the list of nearby locations is correct.

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.