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

Launch portfolios #841

Merged
merged 13 commits into from
Aug 22, 2023
Merged

Launch portfolios #841

merged 13 commits into from
Aug 22, 2023

Conversation

tiffanychu90
Copy link
Member

@tiffanychu90 tiffanychu90 commented Aug 21, 2023

  • re-run all broken redirects for portfolios: atp, competitive_corridors, dla, quarterly_performance_metrics
  • redirects have a problem, but only if we use 200 or 200!...works ok for 301
  • https://docs.netlify.com/routing/redirects/
  • remove hqta -- large maps seem to cause an issue
  • rerun rt but add it back to _redirect...without it, rt link would be broken. but the combination of checking in portfolio/rt/ files leaves broken links...need to fix something in _redirect still?
  • ahsc can use 200 because it has no notebook...301 doesn't work
  • leave ugly url up for now because it's working and it's better than nothing showing up
  • use python portfolio/portfolio.py index --deploy --prod after all the sites are deployed successfully to check that all sites work

@tiffanychu90 tiffanychu90 added the portfolio work Work related to the analytics portfolio label Aug 21, 2023
@github-actions
Copy link

@github-actions
Copy link

1 similar comment
@github-actions
Copy link

@github-actions
Copy link

2 similar comments
@github-actions
Copy link

@github-actions
Copy link

@tiffanychu90 tiffanychu90 merged commit eb2a3cf into main Aug 22, 2023
5 checks passed
@tiffanychu90 tiffanychu90 deleted the launch-portfolios branch August 22, 2023 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
portfolio work Work related to the analytics portfolio
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant