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

MSPB-308: Check for UTF-8 character in extensions in SmartPBX #468

Merged
merged 5 commits into from
Nov 3, 2022

Conversation

Isaac2600Hz
Copy link
Contributor

No description provided.

@Isaac2600Hz Isaac2600Hz requested review from a team and masmerino13 and removed request for a team October 27, 2022 22:24
app.js Outdated Show resolved Hide resolved
app.js Outdated Show resolved Hide resolved
Copy link
Contributor

@masmerino13 masmerino13 left a comment

Choose a reason for hiding this comment

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

Looks really for me, I just left some comments/suggestions, let me know if there are any questions

app.js Outdated Show resolved Hide resolved
submodules/users/users.js Outdated Show resolved Hide resolved
submodules/users/users.js Outdated Show resolved Hide resolved
submodules/users/users.js Outdated Show resolved Hide resolved
submodules/users/users.js Outdated Show resolved Hide resolved
app.js Outdated Show resolved Hide resolved
Copy link
Contributor

@guillegr123 guillegr123 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!

Copy link
Contributor

@masmerino13 masmerino13 left a comment

Choose a reason for hiding this comment

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

no more comments to add

@Isaac2600Hz Isaac2600Hz merged commit bf4b453 into master Nov 3, 2022
@Isaac2600Hz Isaac2600Hz deleted the MSPB-308 branch November 3, 2022 19:10
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.

3 participants