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.

Sign up for GitHub

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

Jump to bottom

#973: Bug fixes after a report in the comments of PR #973 #981

Merged
merged 3 commits into from
Sep 17, 2023
Merged

#973: Bug fixes after a report in the comments of PR #973 #981

merged 3 commits into from
Sep 17, 2023

Conversation

nielsdrost7
Copy link
Contributor

No description provided.

@nielsdrost7
Copy link
Contributor Author

@Verony-makesIT thanks to the report you made in PR #973 I was able to make this pull-request:
#973 (comment)
Feel free to review it

@nielsdrost7 nielsdrost7 merged commit 1a625af into development Sep 17, 2023
1 check passed
@Verony-makesIT
Copy link
Contributor

@nielsdrost7
I have installed the 1.6.1 beta 3 version (incl. bugfix #973) and I have not noticed any issues related to xss_clean.

@nielsdrost7
Copy link
Contributor Author

Thanks for the update.
If you run into anything that could be related to the update (mostly lots of fields have gotten dxtra checks, so test creating clients, invoice, the usual), just report it on the forums

@Verony-makesIT
Copy link
Contributor

FYI, I previously reported a comment in Slack on the (v1.6.1 beta 3) notification from Clockwiseq regarding an (create user) error during the installation of version 1.6.1 beta 3.

nielsdrost7 added a commit that referenced this pull request Dec 16, 2023
@nielsdrost7
#973: Bug fixes after a report in the comments of PR #973 (#981)
1665d1d 
* #973: Bug fixes after a report in the comments of PR #973: too many xss_clean functions

* #973: Applied xss_clean on too many places. This is the correction

* #973: Quick commit
@nielsdrost7 nielsdrost7 deleted the bugfix/973-bugfixes-beta-3 branch March 10, 2024 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Verony-makesIT Verony-makesIT Awaiting requested review from Verony-makesIT

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@nielsdrost7 @Verony-makesIT