Comments (20)
💰 A crowdfund contribution worth 5.00000 DAI (5.0 USD @ $1.0/DAI) has been attached to this funded issue from @bakkdoor.💰
Want to chip in also? Add your own contribution here.
from eth-sig-util.
Okay, a quick review of #73, it seems to have merely moved the .js
to .ts
file. That's very funny, because JS is valid TS, but it isn't what is intended by this issue, I'll be clarifying the master post to clarify.
from eth-sig-util.
@danfinlay i just finished this issue, can you check it out ? #73
from eth-sig-util.
Hi ppl 👋 You might benefit from enabling TS strict mode since the types, numbers, and binary have to be precise. In the CLI --strict
or "strict": true,
in tsconfig.json.
from eth-sig-util.
Issue Status: 1. Open 2. Started 3. Submitted 4. Done
This issue now has a funding of 75.0 DAI (75.0 USD @ $1.0/DAI) attached to it.
- If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
- Want to chip in? Add your own contribution here.
- Questions? Checkout Gitcoin Help or the Gitcoin Slack
- $101,189.03 more funded OSS Work available on the Gitcoin Issue Explorer
from eth-sig-util.
I would like to start this work. ❤️
from eth-sig-util.
@lazaridiscom, I found this file eccrypto-lite.js under the folder /utils
, it is not referenced in the main module. Should i refactor it?
from eth-sig-util.
Since it appears to be unused, you can ignore that file for your typescript migration, we can remove it in a separate change.
Please note that @lazaridiscom is not a MetaMask team member, and is not in a position to define the terms of this bounty, and is merely being helpful by offering his advice.
Lazaridiscom, please try to avoid situations where external contributors mistake you for a team member.
from eth-sig-util.
@lazaridiscom love the help!, your tips are very helpful.
from eth-sig-util.
@mul53 Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!
- reminder (3 days)
- escalation to mods (6 days)
Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days
from eth-sig-util.
Hey @lazaridiscom just realized with the work i currently have, I won't manage to finish the work, would be better to give someone else the bounty.
from eth-sig-util.
Issue Status: 1. Open 2. Started 3. Submitted 4. Done
Work has been started.
These users each claimed they can complete the work by 266 years, 9 months from now.
Please review their action plans below:
1) alexandrumatei36 has applied to start work (Funders only: approve worker | reject worker).
- implement the quick changes necessary to migrate to TS (add tsc compiler, set up the tsconfig file, modify package.json file)
- make sure building and testing works as expected
2) kichjang has been approved to start work.
I took an initial look at the repository and the preceding PR (#70), which contains pretty much the basic setup to get the ball rolling for any JS to TS conversion project.
The next and most important step there would be to add all the relevant typing information for every API and object that is used by the project. I can see that there's a special variable called TYPED_MESSAGE_SCHEMA
, which would the prime candidate to convert to a proper TypeScript interface.
I expect this work to be done in 2 days. I say 2 instead of 1 just to pad in an extra day for unforeseen difficulties that I may encounter during implementation. To speed things up, can I ask where should I contact the maintainers to have a quicker response while I'm working on it?
3) hrithikgautham has applied to start work (Funders only: approve worker | reject worker).
I know JavaScript and Typescript. I can pull this off.
4) leinadpb has applied to start work (Funders only: approve worker | reject worker).
- Examinate all source code.
- Build models (and a service layer if required)
- Refactor code using SOLID principles
5) nttmai has applied to start work (Funders only: approve worker | reject worker).
I worked with ts for a blockchain project before, i want to do this job
6) nttmai has applied to start work (Funders only: approve worker | reject worker).
I worked with ts for a blockchain project before, i want to do this job
7) michalkotas has applied to start work (Funders only: approve worker | reject worker).
Converting each parts into properly typed (no implicit "any") blocks; define interfaces for data structures; verify if tests gives the same results after conversion.
Learn more on the Gitcoin Issue Details page.
from eth-sig-util.
Hey!
I saw this bounty to convert the JavaScript project to TypeScript.
Is this bounty still open? Is anyone working on it right now? I saw the message from the bot above but I'm not sure so if someone could let me know the status that'd be great.
Cheers.
from eth-sig-util.
So, why #71 is closed? It almost looks good to me?
@mul53 ?
from eth-sig-util.
I should also add that it'd be nice if we can also convert the example into using TypeScript.
from eth-sig-util.
@tunnckoCore what @lazaridiscom said is true, it contains a lot of followup tasks. But it can be done, just time i didn't have on my hands.
from eth-sig-util.
Hey there, back from a winter break. I see #73 has been submitted, but it did bypass the requested Gitcoin worker/approve process, which I suspect is why Chiro gave it 👎.
We're going to review that work since it's done and decide from there, will be back here soon, sorry for the delay and the approve/stopped bounty on Chiro, feel free to re-apply, we'll admit if we decide against this submission.
from eth-sig-util.
Hey guys I might also suggest you refactor and reuse all/most strings and numbers. You can convert everything to constants and you can also leverage Object.freeze({})
to ensure users don't overwrite values.
For example Object.freeze
with things like this: https://github.com/MetaMask/eth-sig-util/blob/master/index.js#L7
Reference a constant & don't keep writing and instantiating the same string:
https://github.com/MetaMask/eth-sig-util/blob/master/index.js#L44
https://github.com/MetaMask/eth-sig-util/blob/master/index.js#L50
https://github.com/MetaMask/eth-sig-util/blob/master/index.js#L59
Pull out weird values and describe them with JSDoc comments:
https://github.com/MetaMask/eth-sig-util/blob/master/index.js#L51
https://github.com/MetaMask/eth-sig-util/blob/master/index.js#L210
Does it really need to use this.
? Is it really tracking internal state and should it? Right now it's kind of an unorthodox implicit class declaration. Maybe it can all be pure functions or classes if it really needs internal state.
https://github.com/MetaMask/eth-sig-util/blob/master/index.js#L34
https://github.com/MetaMask/eth-sig-util/blob/master/index.js#L219
If you think it's helpful you can @-ping me any time, kudos, tip or 👍 or 🤗. https://gitcoin.co/tcrowe
from eth-sig-util.
@danfinlay Do you mind doing the honors and sending in the payment for this bounty?
from eth-sig-util.
⚡️ A tip worth 7.50000 DAI (7.5 USD @ $1.0/DAI) has been granted to @KiChjang for this issue from @danfinlay. ⚡️
Nice work @KiChjang! Your tip has automatically been deposited in the ETH address we have on file.
- $75385.74 in Funded OSS Work Available at: https://gitcoin.co/explorer
- Incentivize contributions to your repo: Send a Tip or Fund a PR
- No Email? Get help on the Gitcoin Slack
from eth-sig-util.
Related Issues (20)
- der.ts:6 Uncaught ReferenceError: Buffer is not defined HOT 2
- Update sign-typed-data.js file to support signTypedData_v4
- Array types are parsed incorrectly HOT 1
- Additional `version` options for `encrypt` and `decrypt` possible?
- Remove bn.js
- signTypedData() return value changed in 5.0.0
- Fix bytes encoding not publish HOT 1
- Throw away tweetnacl and tweetnacl-util
- Dead line of type code? HOT 2
- Remove external references from package meta data - codeload.github.com HOT 14
- Cannot convert a BigInt value to a number HOT 9
- Bool encoding ("0" & "false") discrepancy on TypedDataUtils.eip712Hash
- Production build error: Can't convert a BigInt value to a number HOT 4
- Verifying Contract in eth_signTypedData
- Remove ethjs-util HOT 1
- Different signTypedData results between 5.1.0 and 6.0.1 HOT 2
- tsconfig: Disable `strictNullChecks`, enable `strict`
- tsconfig: Target ES2020 instead of ES2017
- Not working in metamask
- 0x1B5F35aabC5022bc8c4693bC0cbbb2FF28442731
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from eth-sig-util.