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

Issue #34: added feature to add reason while rejecting applicant #63

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

atularvind
Copy link
Contributor

@atularvind atularvind commented May 19, 2020

Issue #34

This PR will add a feature on the enrolments to reject the applicant after adding the reason from the configuration. below are the related screenshots for the feature

1. Configuration menu
screenshot_1

2. Reject reason wizard
screenshot_2

_description = "Application Reject Reason"

name = fields.Char('Reason', required=1)
desc = fields.Text('Description')
Copy link
Contributor

Choose a reason for hiding this comment

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

Please expose the desc field in the UI

@@ -138,6 +138,9 @@
<field name="identity_national"/>
<field name="identity_passport"/>
</group>
<group string="Rejection" name="rejection" attrs="{'invisible': [('reject_reason_id', '=', False)]}">
<field name="reject_reason_id"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

expose the rejection note

_name = "wiz.openg2p.applicant.reject.reason"
_description = "Add Reject Reason For Application "

reject_reason_id = fields.Many2one('openg2p.applicant.reject.reason', "Reject Reason", required=1)
Copy link
Contributor

Choose a reason for hiding this comment

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

have rejection note also

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