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

add vebetterdao ids #316

Merged
merged 4 commits into from
Jan 16, 2025
Merged

add vebetterdao ids #316

merged 4 commits into from
Jan 16, 2025

Conversation

mfrezzati
Copy link
Contributor

No description provided.

Copy link

⚠️ Please submit only one app at a time

@libotony
Copy link
Member

Looks like you are adding new parameters like this , could you also update the doc and the validation script?

Copy link

⚠️ Please do not modify other files while submitting an app

@mfrezzati
Copy link
Contributor Author

@libotony Docs updated

README.md Outdated
@@ -76,6 +77,7 @@ Create a `manifest.json` file includes app details.
- `isVeWorldSupported` is **required**, set this to true if your dApp supports the VeWorld browser extension, enabling its use within the VeWorld mobile wallet. Your dApp will then be listed in the dApp section of the VeWorld mobile discovery.
- `repo` is _optional_, and must be a fully-qualified URL.
- `contracts` is _optional_, should be **_an array_** and **_lower cases_** if provided
- `veBetterDaoId` is **required** for VeBetterDAO DApps
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should be optional considering there are VeBetterDAO and Non VeBetterDAO apps.

Copy link

⚠️ Please do not modify other files while submitting an app

@mfrezzati
Copy link
Contributor Author

@libotony changed veBetterDaoId to optional

@libotony
Copy link
Member

There is one last stuff to work with, which is to update validation script for the new filed. I would appreciate if you can work on that, if you don't have bandwidth I am happy to help.

@mfrezzati
Copy link
Contributor Author

mfrezzati commented Jan 16, 2025

@libotony I can work on the validation script, but I'm not sure about the specific validation rules required for the VeBetterDAO ID. Can you help me with this??

Maybe something like this:

function isValid(id: string) { const re = /^0x[a-fA-F0-9]{64}$/; return re.test(hash); }

@libotony
Copy link
Member

@libotony I can work on the validation script, but I'm not sure about the specific validation rules required for the VeBetterDAO ID. Can you help me with this??

Maybe something like this:

function isValid(id: string) { const re = /^0x[a-fA-F0-9]{64}$/; return re.test(hash); }
After checking the code, seems easy, let me do a quick commmit.

Copy link

⚠️ Please do not modify other files while submitting an app

Copy link

@mfrezzati mfrezzati merged commit e5f3be8 into master Jan 16, 2025
6 of 7 checks passed
@libotony libotony deleted the add-vebetterdao-ids branch January 17, 2025 07:02
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