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

WordCloudBlock Extraction #34840

Open
Tracked by #34827
farhan opened this issue May 22, 2024 · 2 comments · May be fixed by openedx/xblocks-contrib#4
Open
Tracked by #34827

WordCloudBlock Extraction #34840

farhan opened this issue May 22, 2024 · 2 comments · May be fixed by openedx/xblocks-contrib#4
Assignees

Comments

@farhan
Copy link
Contributor

farhan commented May 22, 2024

Parent Ticket

@farhan farhan self-assigned this May 22, 2024
@irtazaakram irtazaakram moved this to 🔖 Ready in Aximprovements Team May 22, 2024
@farhan farhan moved this from 🔖 Ready to 🆕 New in Aximprovements Team May 22, 2024
@feanil feanil moved this from 🆕 New to 🔖 Ready in Aximprovements Team May 23, 2024
@farhan farhan moved this from 🔖 Ready to 🏗 In progress in Aximprovements Team Jun 12, 2024
@farhan farhan changed the title WordCloudBlock Extraction R&D WordCloudBlock Extraction Jun 12, 2024
@farhan
Copy link
Contributor Author

farhan commented Jun 12, 2024

@kdmccormick @feanil @irtazaakram Need thoughts

WordCloud XBlock is using EditingMixin

How should we extract down its functionality into separate repo.

  1. Should we extract down the word_cloud xblock related functionality only and put in the xblock repo?
  2. Should we extract down the EditingMixin perhaps and put it into xblock.utils.mixin of xblock repo?

@irtazaakram
Copy link
Member

For WordCloud XBlock I don't think we need EditingMixin. We can create studio view for WordCloud through studioeditablexblockmixin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🏗 In progress
Development

Successfully merging a pull request may close this issue.

2 participants