Skip to content

Conversation

@acpMicrocom
Copy link

No description provided.

@legalsylvain
Copy link
Contributor

legalsylvain commented Sep 19, 2024

thanks for your contribution

/ocabot migration website_slides

Depends on :


@openupgrade.migrate()
def migrate(env, version):
openupgrade.load_data(env, "website_slides", "17.0.2.7/noupdate_changes.xml")
Copy link
Contributor

Choose a reason for hiding this comment

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

delete template translations

website_slides / slide.channel / visibility (selection) : selection_keys is now '['connected', 'members', 'public']' ('['members', 'public']')
# NOTHING TO DO

website_slides / slide.channel.partner / active (boolean) : NEW hasdefault: default
Copy link
Contributor

Choose a reason for hiding this comment

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

better to precreate and fill

website_slides / slide.channel.partner / active (boolean) : NEW hasdefault: default
website_slides / slide.channel.partner / completed (boolean) : DEL
website_slides / slide.channel.partner / last_invitation_date (datetime): NEW
website_slides / slide.channel.partner / member_status (selection) : NEW required, selection_keys: ['completed', 'invited', 'joined', 'ongoing']
Copy link
Contributor

Choose a reason for hiding this comment

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

needs to fill using completion field:

  • 'joined' : enrolled member, not started (completion == 0)
  • 'ongoing' : enrolled member, started but never finished (0 < completion < 100)
  • 'completed' : enrolled member who completed the course (completion == 100)

'invited' is new feature.

Comment on lines +68 to +69
DEL ir.rule: website_slides.rule_slide_channel_not_website (noupdate)
DEL ir.rule: website_slides.rule_slide_slide_not_website (noupdate)
Copy link
Contributor

Choose a reason for hiding this comment

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

safely delete

@MiquelRForgeFlow
Copy link
Contributor

Superseded by #4838.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants