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

EV-2500: fix-sharepoint-online-apostrophes-in-file-names #2

Merged

Conversation

klondikemarlen
Copy link

@klondikemarlen klondikemarlen commented Jun 25, 2021

EV-2500

Overview

Upload a file on Edge to SharePoint Online with an apostrophe in the filename.
The routing will enter a broken state and 500 whenever sync is run.

Solution

🪲 Hopefully fix apostrophes woes.
Fix from Plaristote#39

Testing Instructions

See https://github.com/proofgov/proofgov/wiki/Development:-SharePoint-Docs-Snippets for setting up a SharePoint environment

  1. Checkout https://github.com/proofgov/proofgov/pull/2246 and do a dev bash -> bundle install

Will need to create a Personal Access Token on github and use in place of password for https connection. See https://docs.github.com/en/github/authenticating-to-github/keeping-your-account-and-data-secure/creating-a-personal-access-token

  1. Do a ENABLE_SSL=true dev restart
  2. Go to a SharePoint Online backed routing and upload a previewable file with an apostrophe in the file name.
  3. Click on the link and make sure it loads correctly
  4. Go to a SharePoint Online backed routing and upload a non-previewable file with an apostrophe in the file name.
  5. Click on the link and make sure it loads correctly
  6. Check that upload sync works (click the sync button).
  7. Repeat the process for a SharePoint 2013 backed routing. You will need to create a new routing for this.

Pre-Merge Checklist

  • The work here reflects the user stories in the JIRA ticket(s) listed at the top of this pull request
  • This pull request has been referenced via URL in the JIRA ticket(s) listed
  • I have performed a self-review of the code I have written here
  • I have done end-to-end QA testing of sections of the application I have altered
  • My changes generate no new warnings / errors in the console

Special Pre- or Post-Deployment Instructions

Will require updating and republishing the proofgov/sharepoint-api gem.

Reminder that deploy instructions are here

@klondikemarlen klondikemarlen added the bug Something isn't working label Jun 25, 2021
@klondikemarlen klondikemarlen self-assigned this Jun 25, 2021
@klondikemarlen klondikemarlen marked this pull request as ready for review June 25, 2021 23:12
Copy link

@jaemo jaemo left a comment

Choose a reason for hiding this comment

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

👍🏼

@klondikemarlen klondikemarlen merged commit 91ea482 into master Jun 28, 2021
@klondikemarlen klondikemarlen deleted the EV-2500/fix-sharepoint-online-apostrophes-in-file-names branch June 28, 2021 18:11
@jgigault-pa
Copy link

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants