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

feat: updated po workflow #3780

Merged
merged 4 commits into from
Nov 12, 2024
Merged

feat: updated po workflow #3780

merged 4 commits into from
Nov 12, 2024

Conversation

ibtesam-onefm
Copy link
Collaborator

Is this a Feature, Chore or Bug?

  • Feature
  • [] Chore
  • [] Bug

Clearly and concisely describe the feature, chore or bug.

Need amendments in PO workflow

Analysis and design (optional)

Analyse and attach the design documentation

Solution description

Describe your code changes in detail for reviewers.

  • Removed Purchase Order Approver
  • Updated Workflow according to new process

Is there a business logic within a doctype?

- [] Yes
- [x] No

Output screenshots (optional)

Post the output screenshots, if a UI is affected or added due to this feature.

Areas affected and ensured

List out the areas affected by your code changes.
Purchase Order Workflow

Is there any existing behavior change of other features due to this code change?

Mention Yes or No. If Yes, provide the appropriate explanation.
Only PO workflow is affected

Did you test with the following dataset?

  • Existing Data
  • New Data

Was child table created?

- [] is attachment required?
    did you test attachment

Did you delete custom field?

- [x] Yes
- [] No
    If yes, did you write a delete patch?

Yes, I wrote a patch to delete custom fields and also tested it

Is patch required?

  • Yes
  • [] No

    Was the patch test?

Yes

Which browser(s) did you use for testing?

  • Chrome
  • [] Safari
  • [] Firefox

@Talleyrand333 Talleyrand333 merged commit 982ccaa into staging Nov 12, 2024
2 checks passed
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