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

Fix/rpa ssr #316

Closed
wants to merge 6 commits into from
Closed

Fix/rpa ssr #316

wants to merge 6 commits into from

Conversation

chuanlin2018
Copy link
Collaborator

@chuanlin2018 chuanlin2018 commented May 26, 2023

This checkin fixes two problems:

  1. The Opps error when RPA datacart starts - server side rendering problem.
  2. The Opps error after user confirms a bundle download - caused by missing resTitle.

I disable the cartService.getRpaCart() call for server side rendering. Also use "No title..." if resTitle is not available.

Tested in local but not in docker. I need some time to install python 2.7 in my machine (currently running python3).

@RayPlante
Copy link
Collaborator

This should have been made a PR against the fix/ODD-1083-rpa-datacart; see here for difference with #306.

@chuanlin2018
Copy link
Collaborator Author

I see. I will undo this commit and make change to fix/ODD-1083-rpa-datacart instead. Thanks.

@chuanlin2018 chuanlin2018 mentioned this pull request May 31, 2023
@RayPlante
Copy link
Collaborator

All changes that are in this PR are also in #306; closing this one without merging.

@RayPlante RayPlante closed this May 31, 2023
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

Successfully merging this pull request may close these issues.

2 participants