Skip to content

Purchase order docs v2.12#197

Open
aimee-mcneil-melville wants to merge 5 commits intostagingfrom
purchse-orders-2.12
Open

Purchase order docs v2.12#197
aimee-mcneil-melville wants to merge 5 commits intostagingfrom
purchse-orders-2.12

Conversation

@aimee-mcneil-melville
Copy link
Contributor

First pass on the purchase order docs

Haven't addressed any relationship to goods received other than the goods received tab

Copy link
Contributor

@mark-prins mark-prins left a comment

Choose a reason for hiding this comment

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

thanks @aimee-mcneil-melville !! there's a lot of content here ❤️

have only started to look.. will finish up tomorrow. have made a sneaky commit to order the docs menu correctly too

Comment on lines 54 to 58
2. The list can display a fixed number of purchase orders per page. On the bottom left corner, you can see how many purchase orders are currently displayed on your screen.

3. If you have more purchase orders than the current limit, you can navigate to the other pages by clicking on the page number or using the right of left arrows (bottom right corner).

4. You can also select a different number of rows to show per page using the option at the bottom right of the page.
Copy link
Contributor

Choose a reason for hiding this comment

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

we can remove this bit - we should add a general 'how to use tables' section instead 🙏

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agreed, that would be nice especially with the upcoming table changes!

This section is in many pages currently, so a lot of this duplication will be able to be removed then too 😁

Copy link
Contributor

@mark-prins mark-prins left a comment

Choose a reason for hiding this comment

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

thanks @aimee-mcneil-melville this is a huge job!!

I have some requested changes; given the size of the page I've only run through for a sanity check. haven't gone too far into detail.. but I think this is fine to merge once the edits are done 🙏

Comment on lines 44 to 51
### Go to the Purchase Orders Menu

First you need to have purchase orders enabled for your store.

1. Go to the [Manage](/docs/manage/facilities/#store-preferences) menu in the navigation panel
2. Click on `Stores`, then select your store
3. In the `Preferences` section, enable `Use procurement functionality`

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe the preference mentions aren't needed here, since the preference section is now directly above this?

Copy link
Contributor

Choose a reason for hiding this comment

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

yes good catch, we can remove this bit now

@aimee-mcneil-melville
Copy link
Contributor Author

Thanks for the review Mark!

I've made the changes as suggested, and added a tiny intro sentence at the beginning - I had just written 'Intro'.

Can either expand on that now or later on!

Copy link
Contributor

@mark-prins mark-prins left a comment

Choose a reason for hiding this comment

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

👍 thanks Aimee - that's great thanks for the changes
The only problem is timing.. we should hold off on the merge till we have POs publicly visible

Comment on lines 44 to 51
### Go to the Purchase Orders Menu

First you need to have purchase orders enabled for your store.

1. Go to the [Manage](/docs/manage/facilities/#store-preferences) menu in the navigation panel
2. Click on `Stores`, then select your store
3. In the `Preferences` section, enable `Use procurement functionality`

Copy link
Contributor

Choose a reason for hiding this comment

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

yes good catch, we can remove this bit now

@aimee-mcneil-melville
Copy link
Contributor Author

Thanks Mark I have updated, will wait to merge until ready 🙏

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.

2 participants