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

Update per user form data export #1412

Open
sheralim012 opened this issue Oct 5, 2022 · 3 comments · May be fixed by #1529
Open

Update per user form data export #1412

sheralim012 opened this issue Oct 5, 2022 · 3 comments · May be fixed by #1529
Assignees

Comments

@sheralim012
Copy link
Contributor

  • Update headings
  • Add page id column

per-user-form-data-export

@istride
Copy link
Contributor

istride commented Jan 30, 2023

@cbunicef How concerned should we be about automatic processing that might be happening on these data exports? If we insert a new column at the beginning of each row and rename other columns, will that cause problems?

If it might, we can make this change in a less destructive way by:

  • appending the new column (page id) at the end of the row
  • renaming existing columns, but also duplicating them with their original name at the end of the row (for backwards compatibility, in case they are being referred to by name)
  • allow a grace period
  • remove the duplicated columns in a future release

@cbunicef
Copy link
Collaborator

It's possible that it could, we use Power BI quite often and M is super fragile in my experience. Even appending new columns can cause issues.

Could we set a date to change the data export format, and add a text to the page eg "The format for data exports will be changing on X day, please see here for more information." - and we can link to a page on cms-manual. Is that possible?

We would want to also consider incorporating these issues into any change we are making to the data export format:

We can talk about how important these are vs the effort required.

@cbunicef
Copy link
Collaborator

cbunicef commented Feb 1, 2023

Well build in a period where both formats are supported, add the warning text, and then remove the old format after eg 3 months.

We will put this behind a feature flag while it's in development.

@Alihassanc5 Alihassanc5 linked a pull request Feb 9, 2023 that will close this issue
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants