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

WCM-462: toggle single columns on and off #890

Closed
wants to merge 2 commits into from
Closed

WCM-462: toggle single columns on and off #890

wants to merge 2 commits into from

Conversation

stollero
Copy link
Contributor

@stollero stollero commented Oct 17, 2024

Wie wäre der Ablauf? Wir fügen ein neue Spalte hinzu mit der Info toggled, um diese explizit von den bestehenden zu unterscheiden bzw. später nach der Migration den Toggle zu "entfernen".

Ab diesem Zeitpunkt können wir nach dem Schema namespace_name_write und namespace_name_read die Toggle für die einzelnen Spalten aktivieren, Beispiel workflow_published_write.

Diese Toggle sind weiterhin abhängig von unseren übergeordneten Toggle read_metadata_columns und write_metadata_columns.

-> Abhängig auch von der Implementierung in #889

Checklist

  • Was passiert im DAVPropertyConverter ?
  • Documentation
  • Changelog
  • Tests
  • Translations

gif

@stollero stollero marked this pull request as ready for review October 18, 2024 12:21
def is_column_enabled(cls, column, mode):
namespace = column.info.get('namespace')
name = column.info.get('name')
if column.info.get('toggled'):
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oder wir machen es gleich per Default für alle Felder :D

@wosc
Copy link
Member

wosc commented Oct 23, 2024

ersetzt durch #895

@wosc wosc closed this Oct 23, 2024
@wosc wosc deleted the WCM-462 branch October 23, 2024 09:35
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.

2 participants