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

XAF RGS Module #40

Merged
merged 5 commits into from
May 7, 2024
Merged

XAF RGS Module #40

merged 5 commits into from
May 7, 2024

Conversation

andreschenkels
Copy link
Collaborator

No description provided.

@andreschenkels
Copy link
Collaborator Author

@ThijsvOers could you do a test?

@andreschenkels
Copy link
Collaborator Author

@tarteo is it possible to get a new runboat?

There is a module website_payment_recurring_donations_mollie giving some pre-commit errors on the website in the manifest file.

@tarteo tarteo force-pushed the PR-XAF-Export-with-RGS branch from c198be4 to 76d2849 Compare April 24, 2024 08:23
@andreschenkels
Copy link
Collaborator Author

@ThijsvOers this one has the correct runboat you can review this now.

Copy link

@ThijsvOers ThijsvOers left a comment

Choose a reason for hiding this comment

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

Checked and was able to generate the xaf file where the RGS reference code was in field Lead Reference

Copy link
Contributor

@ByteMeAsap ByteMeAsap left a comment

Choose a reason for hiding this comment

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

LGTM!

@andreschenkels andreschenkels requested a review from ThijsvOers May 1, 2024 14:09

def _get_auditfile_template(self):
"""return the qweb template to be rendered"""
return "l10n_nl_rgs_xaf_auditfile_export.auditfile_template"
Copy link
Member

Choose a reason for hiding this comment

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

If you remove primary="True" from the template (auditfile_template) we don't need this. Correct me if I'm wrong

Copy link
Contributor

Choose a reason for hiding this comment

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

@tarteo , you are correct

@tarteo
Copy link
Member

tarteo commented May 7, 2024

Ignoring automatic tests results.

@tarteo tarteo merged commit 5b82353 into 16.0 May 7, 2024
2 of 3 checks passed
@tarteo tarteo deleted the PR-XAF-Export-with-RGS branch May 7, 2024 11:48
@andreschenkels
Copy link
Collaborator Author

Hoi @tarteo kun je deze laatste wijziging terugdraaien. Na de laatste wijziging werkt het dus niet meer.

@tarteo tarteo restored the PR-XAF-Export-with-RGS branch May 14, 2024 08:29
@tarteo tarteo mentioned this pull request May 14, 2024
@tarteo tarteo deleted the PR-XAF-Export-with-RGS branch May 14, 2024 08:38
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.

4 participants