twill icon indicating copy to clipboard operation
twill copied to clipboard

Published 20 hours ago • verified publisher icon area17

Refactor Twill Settings

Open pboivin opened this issue 3 years ago • 6 comments

Summary

A number of issues with Settings have been reported over time. This groups all previous issues under a single one:

  • [ ] https://github.com/area17/twill/issues/606
  • [ ] https://github.com/area17/twill/issues/895
  • [ ] https://github.com/area17/twill/issues/939
  • [ ] https://github.com/area17/twill/issues/590
  • [ ] https://github.com/area17/twill/issues/1342

pboivin avatar Nov 04 '21 21:11 pboivin

This should also include: #1342

  • [x] Allow browser fields in settings

haringsrob avatar Jan 04 '22 07:01 haringsrob

Any progress on some of this issues?

raz-iacob avatar Feb 28 '22 10:02 raz-iacob

Hi @razvaniacob, singleton modules are a much better approach to create settings pages if you are limited by those issues on settings. We're going to tackle them but they are not our highest priority at the moment. They will be right next after the 3.0 release as indicated by the project roadmap.

ifox avatar Feb 28 '22 10:02 ifox

I can't find any documentation on this.

raz-iacob avatar Feb 28 '22 17:02 raz-iacob

It was just merged last week, you can find it here. The docs site will be updated lated today.

ifox avatar Feb 28 '22 17:02 ifox

I think all this comes down to is making some helpers, maybe an interface, a generator command and documentation.

We also have to think about refactoring old settings, or providing a guide to do so. Maybe we can automate, I don't really know.

haringsrob avatar Apr 13 '22 09:04 haringsrob

Closing. this as the implementation is done in https://github.com/area17/twill/pull/1796

haringsrob avatar Sep 21 '22 06:09 haringsrob