Feature/migrate to deno #410

j4qfrost · 2024-11-23T23:56:18Z

Some deno changes:

.js -> .jsx
Dockerfile's updated for deno
rollup options added for react-chart

Upgrades:

react-router-dom
- useHistory -> useNavigate
- Redirect -> Navigate

Linting:

Some refactor suggestions
Indenting
Removed/commented unused code

CI fixes:

nginx conf needs to be changed manually, there doesn't seem to be any rebuilding for that config
Need to figure out where we deploy to for staging, right now the value is hard coded in CI

undelete

Now using go 1.23! Tested this on dev, it works

Prints initial settings as defined in setup.yaml. Added for usability reasons - users keep getting confused about account balance and maximum allowable debt

* add test for HandleHTTPError undelete * more verbose http codes * Update Dockerfile Now using go 1.23! Tested this on dev, it works * Update compose_dev.sh Prints initial settings as defined in setup.yaml. Added for usability reasons - users keep getting confused about account balance and maximum allowable debt * add failure condition test * formatting and test standards --------- Co-authored-by: Osnat Katz Moon <[email protected]>

* Moving test to appropriate location. * Updating, including moving test location. Test not passing due to precision error but generally passing on the terminal up to 6 dits. * market projection handler, not working. * Update market projected probability. * Safe uint conversion. * Updating, adding noop change to push into pipeline. * Update, test passing. * Update, attempting to solve type converstion uint problem. * Adding projected market layout. * Update, still have react JSON parse error but layout added to buyshares. * Market projection updates on buy shares side, upon button press. * Update, change printf to print.

* Drafting out fees deduction functions and capability. * Attempting working version of initialBetFee function. * Draft adding fees. * Update, adding tests. * Adding working test for GetBetsForMarket * Successful fee util test. * Fees added on backend, tests passing. * Updating such that user record submitted. However fee summing not working yet evidently. * Working fees, at least initial fees. * Updating test scenario passing, made more clear. * Update Dockerfile We need to switch to 3.0.14-1~deb12u2 * Removing logging. * Adding fees, including communicating fees on front end. * totalBetCount to userBetCount so as not to misconstrue meaning. * Simplifying function, test passed. * Simplifying naming. * Updating new function name in test * Changing test to got before want convention. * Update marketid variable * Add combined fee structure for more througough test. * Reverting sale amount to 1 share. * Reversing got want --------- Co-authored-by: Osnat Katz Moon <[email protected]>

* fix: fixed the market buttons not showing on mobile * fix: removed trade button on closed market --------- Co-authored-by: Patrick Delaney <[email protected]>

…arkets#298)

) * Make EconomicsConfig from component structs Auto initialize config Add support to inject config Add support for testing the setup through test doubles Use new fee for placing bets (EachBetFee)

* add failure condition test * Add test for HandleHTTPError (openpredictionmarkets#278) * add test for HandleHTTPError undelete * more verbose http codes * Update Dockerfile Now using go 1.23! Tested this on dev, it works * Update compose_dev.sh Prints initial settings as defined in setup.yaml. Added for usability reasons - users keep getting confused about account balance and maximum allowable debt * add failure condition test * formatting and test standards --------- Co-authored-by: Osnat Katz Moon <[email protected]> * add failure condition test (openpredictionmarkets#282) * Creation of Market Price Projection API (openpredictionmarkets#277) * Moving test to appropriate location. * Updating, including moving test location. Test not passing due to precision error but generally passing on the terminal up to 6 dits. * market projection handler, not working. * Update market projected probability. * Safe uint conversion. * Updating, adding noop change to push into pipeline. * Update, test passing. * Update, attempting to solve type converstion uint problem. * Adding projected market layout. * Update, still have react JSON parse error but layout added to buyshares. * Market projection updates on buy shares side, upon button press. * Update, change printf to print. * Create SECURITY.md * Adding Fees to Buying and Selling Bets (openpredictionmarkets#287) * Drafting out fees deduction functions and capability. * Attempting working version of initialBetFee function. * Draft adding fees. * Update, adding tests. * Adding working test for GetBetsForMarket * Successful fee util test. * Fees added on backend, tests passing. * Updating such that user record submitted. However fee summing not working yet evidently. * Working fees, at least initial fees. * Updating test scenario passing, made more clear. * Update Dockerfile We need to switch to 3.0.14-1~deb12u2 * Removing logging. * Adding fees, including communicating fees on front end. * totalBetCount to userBetCount so as not to misconstrue meaning. * Simplifying function, test passed. * Simplifying naming. * Updating new function name in test * Changing test to got before want convention. * Update marketid variable * Add combined fee structure for more througough test. * Reverting sale amount to 1 share. * Reversing got want --------- Co-authored-by: Osnat Katz Moon <[email protected]> * Frontend fixes (openpredictionmarkets#286) * fix: fixed the market buttons not showing on mobile * fix: removed trade button on closed market --------- Co-authored-by: Patrick Delaney <[email protected]> * Attempting toa ddress cors related security concern. (openpredictionmarkets#298) * Drafting out reporting stats. * Add support for dependency injection to setup (openpredictionmarkets#301) * Make EconomicsConfig from component structs Auto initialize config Add support to inject config Add support for testing the setup through test doubles Use new fee for placing bets (EachBetFee) * remove wsl check * remove watcher --------- Co-authored-by: Osnat Katz Moon <[email protected]> Co-authored-by: Patrick Delaney <[email protected]> Co-authored-by: Marko Kovač <[email protected]> Co-authored-by: Arthur <[email protected]>

* add test for HandleHTTPError undelete * more verbose http codes * Update Dockerfile Now using go 1.23! Tested this on dev, it works * Update compose_dev.sh Prints initial settings as defined in setup.yaml. Added for usability reasons - users keep getting confused about account balance and maximum allowable debt * add failure condition test * formatting and test standards * add frontend file polling for wsl2 * Add test for HandleHTTPError (openpredictionmarkets#278) * add test for HandleHTTPError undelete * more verbose http codes * Update Dockerfile Now using go 1.23! Tested this on dev, it works * Update compose_dev.sh Prints initial settings as defined in setup.yaml. Added for usability reasons - users keep getting confused about account balance and maximum allowable debt * add failure condition test * formatting and test standards --------- Co-authored-by: Osnat Katz Moon <[email protected]> * add failure condition test (openpredictionmarkets#282) * Creation of Market Price Projection API (openpredictionmarkets#277) * Moving test to appropriate location. * Updating, including moving test location. Test not passing due to precision error but generally passing on the terminal up to 6 dits. * market projection handler, not working. * Update market projected probability. * Safe uint conversion. * Updating, adding noop change to push into pipeline. * Update, test passing. * Update, attempting to solve type converstion uint problem. * Adding projected market layout. * Update, still have react JSON parse error but layout added to buyshares. * Market projection updates on buy shares side, upon button press. * Update, change printf to print. * Create SECURITY.md * Adding Fees to Buying and Selling Bets (openpredictionmarkets#287) * Drafting out fees deduction functions and capability. * Attempting working version of initialBetFee function. * Draft adding fees. * Update, adding tests. * Adding working test for GetBetsForMarket * Successful fee util test. * Fees added on backend, tests passing. * Updating such that user record submitted. However fee summing not working yet evidently. * Working fees, at least initial fees. * Updating test scenario passing, made more clear. * Update Dockerfile We need to switch to 3.0.14-1~deb12u2 * Removing logging. * Adding fees, including communicating fees on front end. * totalBetCount to userBetCount so as not to misconstrue meaning. * Simplifying function, test passed. * Simplifying naming. * Updating new function name in test * Changing test to got before want convention. * Update marketid variable * Add combined fee structure for more througough test. * Reverting sale amount to 1 share. * Reversing got want --------- Co-authored-by: Osnat Katz Moon <[email protected]> * Frontend fixes (openpredictionmarkets#286) * fix: fixed the market buttons not showing on mobile * fix: removed trade button on closed market --------- Co-authored-by: Patrick Delaney <[email protected]> * Attempting toa ddress cors related security concern. (openpredictionmarkets#298) * Drafting out reporting stats. * Add support for dependency injection to setup (openpredictionmarkets#301) * Make EconomicsConfig from component structs Auto initialize config Add support to inject config Add support for testing the setup through test doubles Use new fee for placing bets (EachBetFee) * revert * feat(frontend): Compiles frontend into single js file for performance * forgot merge --------- Co-authored-by: Osnat Katz Moon <[email protected]> Co-authored-by: Patrick Delaney <[email protected]> Co-authored-by: Marko Kovač <[email protected]> Co-authored-by: Arthur <[email protected]>

Bumps the npm_and_yarn group with 3 updates in the /frontend directory: [path-to-regexp](https://github.com/pillarjs/path-to-regexp), [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) and [micromatch](https://github.com/micromatch/micromatch). Updates `path-to-regexp` from 1.8.0 to 8.2.0 - [Release notes](https://github.com/pillarjs/path-to-regexp/releases) - [Changelog](https://github.com/pillarjs/path-to-regexp/blob/master/History.md) - [Commits](pillarjs/path-to-regexp@v1.8.0...v8.2.0) Updates `rollup` from 4.12.0 to 4.24.4 - [Release notes](https://github.com/rollup/rollup/releases) - [Changelog](https://github.com/rollup/rollup/blob/master/CHANGELOG.md) - [Commits](rollup/rollup@v4.12.0...v4.24.4) Updates `vite` from 5.1.7 to 5.4.10 - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/v5.4.10/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v5.4.10/packages/vite) Updates `micromatch` from 4.0.5 to 4.0.8 - [Release notes](https://github.com/micromatch/micromatch/releases) - [Changelog](https://github.com/micromatch/micromatch/blob/master/CHANGELOG.md) - [Commits](micromatch/micromatch@4.0.5...4.0.8) --- updated-dependencies: - dependency-name: path-to-regexp dependency-type: direct:production dependency-group: npm_and_yarn - dependency-name: rollup dependency-type: direct:production dependency-group: npm_and_yarn - dependency-name: vite dependency-type: direct:development dependency-group: npm_and_yarn - dependency-name: micromatch dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <[email protected]>

Feature/migrate to deno

j4qfrost · 2024-11-23T23:56:44Z

.github/workflows/deploy-staging.yml

@@ -51,7 +52,7 @@ jobs:
        uses: treosh/lighthouse-ci-action@v12
        with:
          urls: |
-            https://openprediction.xyz/


Need to figure this out.

This was messing me up when testing.

j4qfrost · 2024-11-23T23:57:47Z

data/nginx/vhosts/prod/app.conf.template

+	ssl_protocols TLSv1 TLSv1.1 TLSv1.2 TLSv1.3;
+	http2 on;
+	http3 on;
+	http3_hq on;


Some browser optimizations, need to recompile a custom version of nginx to leverage caching.

j4qfrost · 2024-11-23T23:58:03Z

data/nginx/vhosts/prod/app.conf.template

 	location / {
 		proxy_pass http://frontend:80;
+		add_header Alt-Svc 'h3=":$server_port"; ma=86400';


Needed for http3

j4qfrost · 2024-11-23T23:58:47Z

frontend/src/components/buttons/BaseButton.jsx

-const noButtonHoverStyle = `bg-red-btn-hover hover:bg-red-btn`;
-const neutralButtonStyle = `bg-neutral-btn hover:bg-neutral-btn-hover`;
-const neutralButtonHoverStyle = `bg-neutral-btn-hover hover:bg-neutral-btn`;
+// const yesButtonStyle = `bg-green-btn hover:bg-green-btn-hover`;


… into feature/migrate_to_deno

astrosnat · 2024-12-02T03:11:13Z

frontend/Dockerfile

@@ -1,5 +1,5 @@
 # syntax=docker/dockerfile:1.3-labs
-FROM node:22.4.0
+FROM deno:lastest


deno:lastest doesn't exist, going to change to deno:latest and try again

astrosnat

change to deno.lock rather than package-lock.json

astrosnat · 2024-12-02T07:15:07Z

When trying to build on local, I can now build ok after changing package-lock.json to deno.lock in the Dockerfile, but running ./SocialPredict up I see a websocket connection refused error as well as a file corrupted error for MarketChart.jsx

j4qfrost and others added 30 commits August 26, 2024 05:43

add test for HandleHTTPError

b6d0f3d

undelete

more verbose http codes

d123995

Update Dockerfile

d56d930

Now using go 1.23! Tested this on dev, it works

Update compose_dev.sh

47cb345

Prints initial settings as defined in setup.yaml. Added for usability reasons - users keep getting confused about account balance and maximum allowable debt

add failure condition test

18e7b49

Merge branch 'upstream/main'

c7690d1

Merge remote-tracking branch 'upstream/golang123'

de31424

formatting and test standards

3646abe

Merge branch 'main' of github.com:openpredictionmarkets/socialpredict

0a7fcf0

Merge branch 'main' of github.com:openpredictionmarkets/socialpredict

df26051

add frontend file polling for wsl2

6cbb97c

Merge branch 'main' of github.com:openpredictionmarkets/socialpredict

8f86e23

add failure condition test (openpredictionmarkets#282)

88b4854

Create SECURITY.md

a1e039c

Frontend fixes (openpredictionmarkets#286)

327e906

* fix: fixed the market buttons not showing on mobile * fix: removed trade button on closed market --------- Co-authored-by: Patrick Delaney <[email protected]>

Attempting toa ddress cors related security concern. (openpredictionm…

f10fdde

…arkets#298)

Drafting out reporting stats.

f1b4d43

Add support for dependency injection to setup (openpredictionmarkets#301

4cc882e

) * Make EconomicsConfig from component structs Auto initialize config Add support to inject config Add support for testing the setup through test doubles Use new fee for placing bets (EachBetFee)

Merge branch 'upstream/main'

29eaddf

Merge branch 'main' of github.com:openpredictionmarkets/socialpredict

9692361

Merge branch 'main' of github.com:openpredictionmarkets/socialpredict

61827a6

revert

df4f062

Merge branch 'main' of github.com:openpredictionmarkets/socialpredict

2da9247

feat(ci): Add docker creation workflow

454fa34

separate builds

402bf2d

j4qfrost and others added 13 commits October 5, 2024 12:49

args

82a1481

Merge branch 'main' of github.com:openpredictionmarkets/socialpredict

cf543b4

fix(ci): Staging workflow not setting domain correctly

ffe7f11

refactor(nginx): Whitelist paths

6c3a50d

Merge branch 'fix/staging-workflow-domain'

4043fec

Merge branch 'main' into fix/npm_yarn_updates

6519785

Merge branch 'main' into fix/npm_yarn_updates

77f8370

modify for deno

9b89ab2

Merge pull request #2 from j4qfrost/feature/migrate_to_deno

687f86c

Feature/migrate to deno

allow forks to deploy

08e49d1

allow forks to deploy (#3)

23c949a

fix up CI and nginx conf on deploy machine

c84b7ca

j4qfrost commented Nov 23, 2024

View reviewed changes

j4qfrost added 4 commits November 26, 2024 10:48

Merge branch 'main' of github.com:openpredictionmarkets/socialpredict…

9abb28c

… into feature/migrate_to_deno

whoops

30ed1d0

Merge branch 'main' of github.com:openpredictionmarkets/socialpredict…

d8695b7

… into feature/migrate_to_deno

update lock

1c3ee23

j4qfrost requested a review from pwdel December 1, 2024 01:35

astrosnat requested changes Dec 2, 2024

View reviewed changes

j4qfrost added 3 commits December 1, 2024 19:51

Update Dockerfile

939ab4a

correct dev dockerfile

3a3bc5e

easier to use

e153699

astrosnat requested changes Dec 2, 2024

View reviewed changes

j4qfrost added 2 commits December 28, 2024 23:00

udpate react-charts

79843d6

Merge branch 'main' into feature/migrate_to_deno

20f7588

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/migrate to deno #410

Feature/migrate to deno #410

j4qfrost commented Nov 23, 2024

j4qfrost Nov 23, 2024

j4qfrost Nov 23, 2024

j4qfrost Nov 23, 2024

j4qfrost Nov 23, 2024

j4qfrost Nov 23, 2024

astrosnat Dec 2, 2024

astrosnat left a comment

astrosnat commented Dec 2, 2024

Feature/migrate to deno #410

Are you sure you want to change the base?

Feature/migrate to deno #410

Conversation

j4qfrost commented Nov 23, 2024

j4qfrost Nov 23, 2024

Choose a reason for hiding this comment

j4qfrost Nov 23, 2024

Choose a reason for hiding this comment

j4qfrost Nov 23, 2024

Choose a reason for hiding this comment

j4qfrost Nov 23, 2024

Choose a reason for hiding this comment

j4qfrost Nov 23, 2024

Choose a reason for hiding this comment

astrosnat Dec 2, 2024

Choose a reason for hiding this comment

astrosnat left a comment

Choose a reason for hiding this comment

astrosnat commented Dec 2, 2024