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

Document the ProductOpener::Mail.pm module #5087

Merged
merged 2 commits into from
Apr 2, 2021

Conversation

roshnaeem
Copy link
Contributor

Description: Updated the Mail.pm file to add its documentation in perlpod format.

Fixes: #5084

Related Issue: #2203


=head3 Arguments

Two scalar varaibles that contain the email subject and body are passed as an argument.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Two scalar varaibles that contain the email subject and body are passed as an argument.
Two scalar variables that contain the email subject and body are passed as an argument.


=head3 Arguments

Two scalar varaibles that contain the email subject and body are passed as an argument.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Two scalar varaibles that contain the email subject and body are passed as an argument.
Two scalar variables that contain the email subject and body are passed as an argument.


=head2 send_email_to_producers_admin( SUBJECT, TEXT )

C<send_email_to_producers_admin()> sends email from the contacts email of Open Food Facts to producers admin.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
C<send_email_to_producers_admin()> sends email from the contacts email of Open Food Facts to producers admin.
C<send_email_to_producers_admin()> sends email from the contact email of Open Food Facts to producers admin.


=head3 Arguments

The first argument is a hash reference. The other two arguments are scalar varaibles that consist of email subject and body.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
The first argument is a hash reference. The other two arguments are scalar varaibles that consist of email subject and body.
The first argument is a hash reference. The other two arguments are scalar variables that consist of email subject and body.

Copy link
Member

@CharlesNepote CharlesNepote left a comment

Choose a reason for hiding this comment

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

Thanks! I fixed small typos.

@roshnaeem
Copy link
Contributor Author

Thanks! I fixed small typos.

Thank you so much for your feedback!! I have incorporated changes. Please have a look!

@sonarcloud
Copy link

sonarcloud bot commented Apr 1, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Contributor

@stephanegigandet stephanegigandet left a comment

Choose a reason for hiding this comment

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

Looks good to me, thank you!

Copy link
Member

@CharlesNepote CharlesNepote left a comment

Choose a reason for hiding this comment

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

All right!

@stephanegigandet stephanegigandet merged commit a0c1959 into openfoodfacts:master Apr 2, 2021
@roshnaeem roshnaeem deleted the pod branch April 2, 2021 16:23
@teolemon teolemon added the 📚 Documentation Documentation issues improve the project for everyone. label Aug 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📚 Documentation Documentation issues improve the project for everyone.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document the ProductOpener::Mail.pm module in perlpod format
4 participants