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.

Sign up for GitHub

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

Jump to bottom

A user guide for the factory pattern. #983

Merged
merged 4 commits into from
Mar 6, 2024
Merged

A user guide for the factory pattern. #983

merged 4 commits into from
Mar 6, 2024

Conversation

td202
Copy link
Contributor

@td202 td202 commented Mar 1, 2024 •
edited
Loading

Purpose

Closes #982

This adds a user guide on how (and when not) to use the factory pattern for smart contracts.

Changes

  • Add factory pattern guide.

Checklist

  • My code follows the style of this project.
  • The code compiles without warnings.
  • I have performed a self-review of the changes.
  • I have documented my code, in particular the intent of the
    hard-to-understand areas.
  • (If necessary) I have updated the CHANGELOG.

@td202 td202 requested review from abizjak, DOBEN and dg-concordium March 1, 2024 17:17
@DOBEN DOBEN added the preview Trigger a documentation preview for a PR. label Mar 4, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 4, 2024 •
edited
Loading

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-03-06 14:09 UTC

DOBEN
DOBEN approved these changes Mar 4, 2024
View reviewed changes
source/mainnet/smart-contracts/guides/contract-dev-guides.rst Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
dg-concordium
dg-concordium approved these changes Mar 4, 2024
View reviewed changes
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
source/mainnet/smart-contracts/guides/factory-pattern.rst Outdated Show resolved Hide resolved
@td202 td202 merged commit d1bf378 into main Mar 6, 2024
2 checks passed
@td202 td202 deleted the factory-pattern branch March 6, 2024 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abizjak abizjak abizjak approved these changes

@DOBEN DOBEN DOBEN approved these changes

@dg-concordium dg-concordium dg-concordium approved these changes

Assignees
No one assigned
Labels
preview Trigger a documentation preview for a PR.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Expose Module Reference: Factory pattern guide
4 participants
@td202 @abizjak @DOBEN @dg-concordium