Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update ups access point delivery article #296

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

MaWeXi
Copy link

@MaWeXi MaWeXi commented Jul 5, 2024

With the new interface we need to update the article for UPS PUDO shipments and delete the article for ups access point notifications as it is no longer supported.

remove the following lines:

  • "you have to provide the exact address of the UPS access point using the drop_off_point attribute"
  • "you also have to use the additional service ups_access_point_notification [see example]"

add the following lines to the remaining lines:

  • "the recipients address has to be provided as you normally would using the to attribute"
  • "to.first_name is mandatory"
  • "to.last_name is mandatory"
  • "to.email is mandatory"
  • "drop_off_point.drop_off_point_id is mandatory and has to be 9 characters long (can be determined through our pudo provider)"
  • "drop_off_point.type has to be parcel_shop"

sc-30429

Comment on lines 29 to 33
<code><em>drop_off_point.drop_off_point_id</em></code> is manadatory and has to be 9 characters long (can be determined through our
<a style="white-space: nowrap;" href="{{ site.baseurl }}/swagger-ui/#/default/get_pickup_dropoff_locations">
pudo provider
</a>
)
Copy link
Author

Choose a reason for hiding this comment

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

This produces an unwanted whitespace which I do not know how to fix, like this:
(can be determined through our pudo provicer )

Choose a reason for hiding this comment

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

Could you put everything in one line? Would that fix that?

<a style="white-space: nowrap;" href="{{ site.baseurl }}/swagger-ui/#/default/get_pickup_dropoff_locations">pudo provider</a>)

Copy link
Author

Choose a reason for hiding this comment

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

It did fix the problem, thank you.

@MaWeXi MaWeXi requested a review from SteffenHam July 25, 2024 09:31
Copy link

@SteffenHam SteffenHam left a comment

Choose a reason for hiding this comment

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

RTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants