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

force edit for assistants and admins #169

Open
TheMoksej opened this issue Oct 23, 2020 · 8 comments
Open

force edit for assistants and admins #169

TheMoksej opened this issue Oct 23, 2020 · 8 comments
Assignees
Labels
feature enhancement Issue/PR related to a feature implementation, etc. low priority Issue/PR is low priority.
Milestone

Comments

@TheMoksej
Copy link

Ability for assistants and admins to edit bots forcefully.

Let's say the privacy policy doesn't exist, or long description is too short - we'd get an error when trying to edit, however force edit would bypass those checks and let us edit those bots.

we could just contact the owner, but what if they're not in the server and everything is okay except the long description being too short because they have iframe in there.

@TheMoksej TheMoksej added the feature enhancement Issue/PR related to a feature implementation, etc. label Oct 23, 2020
@carolinaisslaying
Copy link
Member

@advaith1 We could make it ignore the desc limit if it has iframe in it?

@advaith1
Copy link
Member

oh did we not add that yet? I'll add that today

@carolinaisslaying
Copy link
Member

carolinaisslaying commented Oct 26, 2020

oh did we not add that yet? I'll add that today

Did you add that? If not I can add it right now.

@advaith1
Copy link
Member

added in c008e84

@carolinaisslaying
Copy link
Member

added in c008e84

Would you say this feature request is redundant now then?

@TheMoksej
Copy link
Author

added in c008e84

Would you say this feature request is redundant now then?

I mean just in case, if privacy policy doesn't exist or bot was submitted before we added the long description limit, sure, why not

@carolinaisslaying
Copy link
Member

added in c008e84

Would you say this feature request is redundant now then?

I mean just in case, if privacy policy doesn't exist or bot was submitted before we added the long description limit, sure, why not

Seems unnecessary imo, I can always make a db request /shrug

@advaith1
Copy link
Member

no point adding UI but I could add a query param or something because why not

@carolinaisslaying carolinaisslaying added the low priority Issue/PR is low priority. label Oct 28, 2020
@carolinaisslaying carolinaisslaying self-assigned this Apr 8, 2022
@carolinaisslaying carolinaisslaying added this to the 5.5.0 milestone Apr 8, 2022
@carolinaisslaying carolinaisslaying modified the milestones: 5.5.0, 6.0.0 May 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature enhancement Issue/PR related to a feature implementation, etc. low priority Issue/PR is low priority.
Projects
None yet
Development

No branches or pull requests

3 participants