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

refactor: contact --> sender in PXE API #10861

Merged

Conversation

benesjan
Copy link
Contributor

registerContact has been deemed confusing because the name is too similar to registerContract.
For this reason we've decided to rename it.

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@benesjan benesjan force-pushed the 12-18-refactor_pxe.registercontact_--_pxe.registersender branch from 20d9273 to f18e38a Compare December 18, 2024 18:52
@benesjan benesjan changed the title refactor: PXE.registerContact --> PXE.registerSender refactor: contact --> sender in PXE API Dec 18, 2024
@AztecBot
Copy link
Collaborator

AztecBot commented Dec 18, 2024

Docs Preview

Hey there! 👋 You can check your preview at https://676464315779431724fedcd4--aztec-docs-dev.netlify.app

@benesjan benesjan requested a review from Thunkar December 18, 2024 20:38
@iAmMichaelConnor
Copy link
Contributor

Pasted from slack:
It's technically not just registering a sender, though. It's also registering a potential recipient. You're going to receive things and send things to this person.
registerSender is misleading in a different way.

registerCounterparty? registerConnection? registerFriend? registerPerson? registerSomeone?

@benesjan benesjan removed the request for review from Thunkar December 18, 2024 21:07
@benesjan benesjan marked this pull request as draft December 18, 2024 21:07
@benesjan benesjan marked this pull request as ready for review December 19, 2024 15:03
@benesjan benesjan requested a review from Thunkar December 19, 2024 15:03
Copy link
Contributor

@rahul-kothari rahul-kothari left a comment

Choose a reason for hiding this comment

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

LGTM - please merge master's migration notes!

@@ -6,6 +6,21 @@ keywords: [sandbox, aztec, notes, migration, updating, upgrading]

Aztec is in full-speed development. Literally every version breaks compatibility with the previous ones. This page attempts to target errors and difficulties you might encounter when upgrading, and how to resolve them.

## TBD
Copy link
Contributor

Choose a reason for hiding this comment

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

can you merge with master and include these in the 0.68.0 section? 🙏

@benesjan benesjan force-pushed the 12-18-refactor_pxe.registercontact_--_pxe.registersender branch from f18e38a to c40ee3a Compare December 19, 2024 18:02
@benesjan benesjan enabled auto-merge (squash) December 19, 2024 18:02
@benesjan benesjan merged commit 8cb26e9 into master Dec 19, 2024
50 checks passed
@benesjan benesjan deleted the 12-18-refactor_pxe.registercontact_--_pxe.registersender branch December 19, 2024 18:37
TomAFrench added a commit that referenced this pull request Dec 20, 2024
* master: (119 commits)
  chore(master): Release 0.68.0 (#10834)
  feat: enable profiling with local PXE in cli-wallet (#10736)
  chore: values for sepolia deployment (#10362)
  fix: vm_full_tests.yml (#10912)
  chore: disable bb-sanitizers.yml (#10901)
  chore: Check yarn version during bootstrap (#10910)
  fix(p2p): default peer score penalties (#10896)
  feat: Updated metrics (#10885)
  feat: Reduce bundle sizes, remove polyfills (#10877)
  chore: Replace bbup.dev link (#10908)
  chore(avm): extra column information in lookups (#10905)
  git subrepo push --branch=master noir-projects/aztec-nr
  git_subrepo.sh: Fix parent in .gitrepo file. [skip ci]
  chore: replace relative paths to noir-protocol-circuits
  git subrepo push --branch=master barretenberg
  chore: disable flake in e2e_fees/private_payments (#10898)
  chore(ci): disable e2e_cheat_codes.test.ts (#10897)
  fix: increase default heartbeat (#10891)
  fix(p2p): less verbose error (#10886)
  refactor: `contact` --> `sender` in PXE API (#10861)
  ...
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.

4 participants