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

Handle not set, optional properties as false #53

Open
derpeter opened this issue Sep 18, 2018 · 1 comment
Open

Handle not set, optional properties as false #53

derpeter opened this issue Sep 18, 2018 · 1 comment
Assignees
@derpeter
Labels
enhancement
Milestone
36c3

Comments

@derpeter
Copy link
Contributor

derpeter commented Sep 18, 2018 •
edited

Instead of requering e.g. Publishing.Mastodon.Enable to be set even if we dont want it we should handle this as false. This would reduce the amount of properties needed to be set.
This should be changed for:

  • Twitter
  • Mastodon
  • Voctoweb
  • Youtube
@derpeter derpeter self-assigned this Sep 18, 2018
@derpeter derpeter added this to the 35c3 milestone Sep 18, 2018
@derchrisuk
Copy link
Member

Yes, I would only explicitly enable them if required, otherwise leave them disabled.

@derpeter derpeter modified the milestones: 35c3, camp19 Jan 17, 2019
@derpeter derpeter modified the milestones: camp19, 35c3 Oct 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@derpeter derpeter

Labels
enhancement
Projects
None yet
Milestone
36c3
Development

No branches or pull requests
2 participants
@derpeter @derchrisuk