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

Unexpected "Copied_from_block" field in Advanced Settings #34370

Open
DmytroAlipov opened this issue Mar 14, 2024 · 4 comments
Open

Unexpected "Copied_from_block" field in Advanced Settings #34370

DmytroAlipov opened this issue Mar 14, 2024 · 4 comments

Comments

@DmytroAlipov
Copy link
Contributor

Description:

I noticed that a new Copied_from_block field has appeared in Advanced Settings.
screen_29

However, having investigated this field, I still could not find an explanation for why it was needed here. At the moment it looks confusing.
Perhaps when adding this functionality, they simply forgot to include copied_from_block in FIELDS_EXCLUDE_LIST.

Suggestion:

Add copy_from_block to the FIELDS_EXCLUDE_LIST and then this field will not be shown in Advanced Settings.

@qasimgulzar
Copy link
Contributor

@DmytroAlipov I am interested in working on this ticket, if you haven't already started working on it.

@DmytroAlipov
Copy link
Contributor Author

Hi @qasimgulzar
You can safely take on this task 👍

@qasimgulzar
Copy link
Contributor

I have investigated and it seems like an internal field, it is safe to put in FIELDS_EXCLUDE_LIST list. I am opening a new PR for this

qasimgulzar added a commit to qasimgulzar/edx-platform that referenced this issue Apr 4, 2024
qasimgulzar added a commit to qasimgulzar/edx-platform that referenced this issue Apr 4, 2024
@qasimgulzar
Copy link
Contributor

Here I have created a PR

qasimgulzar added a commit to qasimgulzar/edx-platform that referenced this issue Apr 8, 2024
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

No branches or pull requests

2 participants