Skip to content
This repository has been archived by the owner on Dec 6, 2021. It is now read-only.

docs: Docs for Custom Webform Context hook #173

Closed

Conversation

shariquerik
Copy link
Member

@shariquerik shariquerik commented Aug 3, 2021

Docs for getting context for custom webform using hooks.

image

Copy link
Member

@surajshetty3416 surajshetty3416 left a comment

Choose a reason for hiding this comment

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

Please update documentation according to recent changes in the feature PR.

@@ -433,6 +433,23 @@ The above configuration will render `/not_found` when a 404 is occurred. It is
upto you to implement the template `www/not_found.html` and controller
`www/not_found.py`.

## Custom Web Form Context
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
## Custom Web Form Context
## Web Form Context

@@ -433,6 +433,23 @@ The above configuration will render `/not_found` when a 404 is occurred. It is
upto you to implement the template `www/not_found.html` and controller
`www/not_found.py`.

## Custom Web Form Context

When a Web Form is created using non custom doctype, we can get the context from doctype's module but for Web Form using custom doctype you cannot get the context since custom doctype don't have modules, you can use this hook to create a context for Custom Web Form.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
When a Web Form is created using non custom doctype, we can get the context from doctype's module but for Web Form using custom doctype you cannot get the context since custom doctype don't have modules, you can use this hook to create a context for Custom Web Form.
When a Web Form is created for a standard doctype, we can get the context from doctype's module. Although for a Web Form for a custom doctype you cannot get the context since custom doctype don't have modules, you can use this hook to create a context for Custom Web Form.

I tried re-wording this.

The sentence is too complex...Maybe break it down into 2-3 simpler sentences.

Use standard instead of "non custom", if it fits.

Also, wording for "doctype's module" is ambiguous. Even I'm not sure what you're talking about 😅

@hasnain2808
Copy link
Contributor

Hi @shariquerik ,

We moved to a new simpler contribution pipeline. Please move your changes (with requested changes) in this PR to the new site.

You can now directly contribute to frappeframework.com. For new contribution guidelines, refer to frappeframework.com/contribute.

Thanks for the contribution!

@hasnain2808 hasnain2808 closed this Dec 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants