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

Dev/17 peridot #1681

Merged
merged 58 commits into from
Mar 16, 2021
Merged

Dev/17 peridot #1681

merged 58 commits into from
Mar 16, 2021

Conversation

iturgeon
Copy link
Member

@iturgeon iturgeon commented Feb 10, 2021

  • #1692 - upgrade katex from 0.11.1 to 0.12.0
  • #1617 - Rectified how highest attempts are marked
  • #1613 - query timeouts added
  • #1608 - Restrict allowed characters in id
  • #1605 - Clearer message for assessment ID error
  • #1603 - makes sure .eslintrc included in eslint-config-obojobo package
  • #1585 - Triple-clicking selection issue
  • #1572 - github test action optimization
  • #1438 - Issue/1234 youtube user friendly

generated by running hub pr list --base dev/17-peridot --state closed --format "- [#%I](%U) - %t%n"

keeeeeegan and others added 30 commits April 8, 2020 15:14
…e and youtube url now gets processed on blur
We had an outage where our AWS RDS database failed over to
its backup server.  Normally one would expect the db driver to lose
connection or at least somehow be aware of the outage.  It wasn't, and just
sat waiting for every db query to resolve.  No errors logged, no automatic
recovery.  The node proccesses needed to be terminated to re-establish
connections.

This commit adds query_timeouts to the configuration for pg-promise to ensure
that they will eventually time out.  We set them just below chromes usual request timeout
to maximize the time before timeout occurs.  This should give us a lot of headroom
for unforseen long-running queries and still produce error logs we can use to indicate
something has gone wrong.

Important: adds a required env var: DB_TIMEOUT, suggested value 29000
@zachberry zachberry had a problem deploying to obojobo-dev-17-peridot-xepiili February 15, 2021 20:33 Failure
makes sure .eslintrc included in eslint-config-obojobo package
@zachberry zachberry had a problem deploying to obojobo-dev-17-peridot-xepiili February 15, 2021 20:35 Failure
@zachberry zachberry temporarily deployed to obojobo-dev-17-peridot-xepiili February 15, 2021 20:35 Inactive
…essment

Rectified how highest attempts are marked
@iturgeon iturgeon temporarily deployed to obojobo-dev-17-peridot-dvr9k2v February 22, 2021 18:20 Inactive
@iturgeon iturgeon temporarily deployed to obojobo-dev-17-peridot-uxm2l5s February 22, 2021 21:12 Inactive
@iturgeon iturgeon temporarily deployed to obojobo-dev-17-peridot-zb4opiz February 22, 2021 21:22 Inactive
…ction-issue

Triple-clicking selection issue
@iturgeon iturgeon temporarily deployed to obojobo-dev-17-peridot-xh1cvif February 23, 2021 15:23 Inactive
@iturgeon iturgeon temporarily deployed to obojobo-dev-17-peridot-fecqgre February 23, 2021 15:32 Inactive
@iturgeon iturgeon temporarily deployed to obojobo-dev-17-peridot-sesrymz February 24, 2021 16:24 Inactive
@iturgeon iturgeon temporarily deployed to obojobo-dev-17-peridot-08ktmo3 February 24, 2021 16:25 Inactive
@iturgeon iturgeon temporarily deployed to obojobo-dev-17-peridot-08ktmo3 February 24, 2021 19:15 Inactive
@iturgeon iturgeon temporarily deployed to obojobo-dev-17-peridot-08ktmo3 February 24, 2021 20:10 Inactive
@iturgeon iturgeon temporarily deployed to obojobo-dev-17-peridot-08ktmo3 February 25, 2021 18:10 Inactive
@iturgeon iturgeon marked this pull request as ready for review March 16, 2021 20:08
@iturgeon iturgeon merged commit 6251f71 into master Mar 16, 2021
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.

7 participants