Skip to content

🛂 Remove some field validators #249

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

🛂 Remove some field validators #249

wants to merge 2 commits into from

Conversation

ff137
Copy link
Collaborator

@ff137 ff137 commented May 15, 2025

Any field validators that may impact cheqd dids, schemas, rev reg ids, etc, are removed for the time being, in order to simplify migration from sov to cheqd. didx-xyz/acapy-cloud#1552

Any field validators that may impact cheqd dids, schemas, rev reg ids, etc, are removed for the time being, in order to simplify migration from sov to cheqd.
didx-xyz/acapy-cloud#1552
ff137 added a commit to didx-xyz/acapy-cloud that referenced this pull request May 21, 2025
* 🧪 Create did:cheqd for governance

* ✨ Support posting did:cheqd schemas to trust registry

* 🚧 Support fetching did:cheqd schemas from TR

Need to accommodate slashes in schema id, which is being passed as a query param

* 🔧 Disable registering sov did for governance

* 🎨 Default cheqd

* 🚧

* 🔧 Disable ledger

* 🚧 Use cloudcontroller with some field validators removed

didx-xyz/aries-cloudcontroller-python#249

* 🐛 TODO: retrieve schema_id from cheqd cred def

* 🐛 Fix reading did from cred def id for cheqd

* 🚧 Use updated debug branch for cheqd plugin

* ♻️ Change sov to cheqd

* 🎨 Rename module

* 🛂 Remove endorser connection check for revocable cred def

* 🎨 Cred def: create_transaction_for_endorser=False

* 🧪 Fix unit tests

* 👷 Dump DID Registrar/Resolver logs

* 🚧 Remove parallel tests command

* 🔧 Reduce default registry size to 5000

* 🔧 Set did-registrar log level to warn

* 🐛 Match `get_schema` on entire path

* 🐛 Replace hack to `get_schema` by matching entire path

* 🔧 Set did-registrar log level to warn

* 🧪 Fix test: issuer no longer has endorser connection

* 🧪 Fix test: remove did:sov

* 🧪 Fix schema id assertion

* 🧪 Skip setting endpoint for cheqd did

* 🐛 Fix registrar log level

* 🧪

* 🧪 Cover cheqd schema

* 🎨

* 🐛 Add additional path extender

* 🐛 Yet another path extender!

* 🧪

* 🚧 Update acapy base image

* 🚧 Update cheqd plugin - improved logging

* 🚧 Use same commit hash for nats and cheqd plugins

* 🔥 Remove Indy Ledger from Local Dev (#1566)

* 🔥 Remove Indy Ledger from Local Dev

* 🥷🦨 Blacksmith 8vCPU runner for local tests

* 🔥 Remove Ledger Nodes from Docker Build + Local Tests (#1567)

* 🔥 Remove Docker Cache Logs

Also disable Mise caching in Local Tests

* 🔥 Disable Python venv cache

* ⏪ Revert "🔥 Disable Python venv cache"

This reverts commit a20fcd1.

* 👷 Use bigger Github Runner rather than Blacksmith

Re-enable Mise Caching

* 🧪 Fix Ed25519 signature version

* 🧪 Skip broken did-exchange/rotate tests for now

* ⬆️ Use latest plugin commit hash

* 👷 Add Cheqd testnet mnemonic to CI/CD (#1569)

Configure the GitHub Actions workflow to pass the Cheqd testnet mnemonic
secret to the `driver-did-cheqd` component. This allows the driver to
interact with the Cheqd testnet blockchain during CI/CD runs and local
tests.

* The mnemonic is used as a fee-payer account for testnet transactions
* Added as a required secret to both `cicd.yml` and `local-tests.yml` workflows
* Passed as `FEE_PAYER_TESTNET_MNEMONIC` environment variable to Tilt

* 🎨 `mise run fmt`

* 🎨 Remove commented out code

* 🎨 Fix return types

* 📝 Update OpenAPI Specs

* 🔥 Delete Indy Ledger in Dev (#1570)

* 🔥 Delete Indy Ledger in Dev

* Delete Indy Ledger from Helmfile and Dev config
* Set `ACAPY_NO_LEDGER` in Dev

* 🔧 Configure Cheqd plugin in Dev

* 🔧 Set default registry size to 200

* K6: with temporary OOB

* 🎨 Remove  from create tenant request

* 🔧 Remove ACAPY_WALLET_SEED from dev governance agent

* 👷 Remove xdist config from pytest

* 🔖 5.0.0-rc2

* K6: use OOB invitation

* K6: Delete issuers at end of tests

* 🔊 Improve error logging in cheqd plugin

* 🎨 Remove unused wallet_type arg

* 🎨 Improve cheqd error messages

* ✨ Support configuring Cheqd network

* 🎨 Remove sov from supported methods

* ✨ 🐛 Add asyncio.lock to DIDRegistrar

* 🎨 Remove wallet type from fixture method

* 🚧 Test cheqd plugin fix

Force 1 second sleep between sequential requests, and retry if account sequence mismatch occurs

* 🎨 Improve error messages when Cheqd resources fail

* 🚧 Debug

* K6: Re-enable revocation

* 🚧 Test queuing pending jobs

* ✅ Update fixtures and tests (#1574)

* add :path to resolve new ids

* update test to make api call

* import router

* use asyncClient

* make api call

* use RichAsyncClient

* make api call

* use RichAsyncClient

* make api call

* use RichAsyncClient

* make api call and respond with model

* use RichAsyncClient

* change scope

* remove import

* raise value error if bad test_mode

* 🚧

* 🐛 Fix indent

* 🚧

* 🚧

* 🚧

* 🔧 Update TESTNET_ENDPOINT for dev

* 🧪 Skip test properly for later review

* 🚧 Attempt to fix lock write bug

* 📝 Update OpenAPI Spec

* 🎨

* 🎨 Remove prints

* 🚧 Attempt to fix retries

* 🔧 GP3 instead of EFS

* 🔧 Increase REGISTRY_CREATION_TIMEOUT

* 🎨 Fix spec

* 🔧 Pod/Container Security Context on MT Agent

* 🔧 Acapy Agent Fix volumes

---------

Co-authored-by: Robbie Blaine <4052340+rblaine95@users.noreply.github.com>
Co-authored-by: Robbie Blaine <robbieblaine.rb@gmail.com>
Co-authored-by: wdbasson <wlmbasson@gmail.com>
Co-authored-by: cl0ete <cloete.dupreez@gmail.com>
Copy link

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.

1 participant