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

fix: missing virtual keyword #452

Merged
merged 6 commits into from
Aug 29, 2023
Merged

Conversation

heueristik
Copy link
Contributor

@heueristik heueristik commented Aug 28, 2023

Description

Task: OS-655

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have selected the correct base branch.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • Any dependent changes have been merged and published in downstream modules.
  • I ran all tests with success and extended them if necessary.
  • I have updated the CHANGELOG.md file in the root folder.
  • I have updated the DEPLOYMENT_CHECKLIST file in the root folder.
  • I have updated the UPDATE_CHECKLIST file in the root folder.

@heueristik heueristik marked this pull request as ready for review August 28, 2023 07:19
@heueristik heueristik force-pushed the feature/OS-655-missing-virtual-keyword branch from 8851d78 to 7943d3e Compare August 28, 2023 07:43
@heueristik heueristik force-pushed the feature/OS-655-missing-virtual-keyword branch from 7943d3e to 6762f26 Compare August 28, 2023 07:44
@heueristik heueristik changed the base branch from develop to main August 28, 2023 07:44
packages/contracts/CHANGELOG.md Outdated Show resolved Hide resolved
@heueristik heueristik merged commit c6f9c8e into main Aug 29, 2023
10 checks passed
@heueristik heueristik deleted the feature/OS-655-missing-virtual-keyword branch August 29, 2023 07:52
heueristik added a commit that referenced this pull request Aug 30, 2023
* fix(subgraph): update to @graphprotocol/[email protected] (#453)

* fix: missing virtual keyword (#452)

* fix: added missing virtual keywords.

* fix: correct misspelled filename

* docs: improved NatSpec

* chore: bump package versions

* Update packages/contracts/CHANGELOG.md

Co-authored-by: Mathias Scherer <[email protected]>

---------

Co-authored-by: Mathias Scherer <[email protected]>

---------

Co-authored-by: Mathias Scherer <[email protected]>
mathewmeconry added a commit that referenced this pull request Oct 6, 2023
* fix(subgraph): update to @graphprotocol/[email protected] (#453)

* fix: missing virtual keyword (#452)

* fix: added missing virtual keywords.

* fix: correct misspelled filename

* docs: improved NatSpec

* chore: bump package versions

* Update packages/contracts/CHANGELOG.md

Co-authored-by: Mathias Scherer <[email protected]>

---------

Co-authored-by: Mathias Scherer <[email protected]>

* feat: deploy v1.3.0-rc0 to sepolia

* feat: bump subgraph version to v1.3.0-rc0
feat: deploy sepolia subgraph

* fix(OS-638): fixes typo in implementation

* chore: fix formatting in helpers script

* chore: fix merge conflict

---------

Co-authored-by: Michael Heuer <[email protected]>
mathewmeconry added a commit that referenced this pull request Nov 30, 2023
* fix(subgraph): update to @graphprotocol/[email protected] (#453)

* fix: missing virtual keyword (#452)

* fix: added missing virtual keywords.

* fix: correct misspelled filename

* docs: improved NatSpec

* chore: bump package versions

* Update packages/contracts/CHANGELOG.md

Co-authored-by: Mathias Scherer <[email protected]>

---------

Co-authored-by: Mathias Scherer <[email protected]>

* Deploy contracts and subgraph to sepolia (#475)

* feat: deploy v1.3.0-rc0 to sepolia

* feat: bump subgraph version to v1.3.0-rc0
feat: deploy sepolia subgraph

* fix(OS-638): fixes typo in implementation

* chore: fix formatting in helpers script

* fix: ipfs returned an empty string (#485) (#495)

* chore: prepared arbitrum deployment

* chore: deployed v1.3.0 to arbitrumGoerli

* docs: improved checklist

* chore: deployed v1.3.0 to arbitrum

* chore: deployed v1.3.0-rc0.4 npm packages

* chore: added manifests for arbitrum and arbitrumGoerli

* docs: improved deployment checklist

* docs: improved deployment checklist

* docs: fix typo

Co-authored-by: Mathias Scherer <[email protected]>

* chore: fix merge

---------

Co-authored-by: Mathias Scherer <[email protected]>
mathewmeconry added a commit that referenced this pull request Dec 12, 2023
* fix(subgraph): update to @graphprotocol/[email protected] (#453)

* fix: missing virtual keyword (#452)

* fix: added missing virtual keywords.

* fix: correct misspelled filename

* docs: improved NatSpec

* chore: bump package versions

* Update packages/contracts/CHANGELOG.md

Co-authored-by: Mathias Scherer <[email protected]>

---------

Co-authored-by: Mathias Scherer <[email protected]>

* Deploy contracts and subgraph to sepolia (#475)

* feat: deploy v1.3.0-rc0 to sepolia

* feat: bump subgraph version to v1.3.0-rc0
feat: deploy sepolia subgraph

* fix(OS-638): fixes typo in implementation

* chore: fix formatting in helpers script

* fix: ipfs returned an empty string (#485) (#495)

* chore: deploy 1.3.0-rc0 to arbitrum and arbitrumGoerli (#496)

* chore: prepared arbitrum deployment

* chore: deployed v1.3.0 to arbitrumGoerli

* docs: improved checklist

* chore: deployed v1.3.0 to arbitrum

* chore: deployed v1.3.0-rc0.4 npm packages

* chore: added manifests for arbitrum and arbitrumGoerli

* docs: improved deployment checklist

* docs: improved deployment checklist

* docs: fix typo

Co-authored-by: Mathias Scherer <[email protected]>

---------

Co-authored-by: Mathias Scherer <[email protected]>

* docs(contracts): removes outdated step in upgrade checklist

* deploy(contracts): adds v1.3.0 mainnet deployment

* deploy(contracts): adds polygon deployment

* fix: removes uncessary inconsistent managingDAOImplementation field from active_contracts.json

* chore: bump package version to v1.3.0

* chore: updates version for osx package

* chore(contracts): updates changelog

* chore(contracts): updates active_contracts with implementations and bases

* chore(contracts): moves line in changelog

* style(contracts): applies prettier styling

---------

Co-authored-by: Michael Heuer <[email protected]>
heueristik added a commit that referenced this pull request Dec 20, 2023
* fix(subgraph): update to @graphprotocol/[email protected] (#453)

* fix: missing virtual keyword (#452)

* fix: added missing virtual keywords.

* fix: correct misspelled filename

* docs: improved NatSpec

* chore: bump package versions

* Update packages/contracts/CHANGELOG.md

Co-authored-by: Mathias Scherer <[email protected]>

---------

Co-authored-by: Mathias Scherer <[email protected]>

* Deploy contracts and subgraph to sepolia (#475)

* feat: deploy v1.3.0-rc0 to sepolia

* feat: bump subgraph version to v1.3.0-rc0
feat: deploy sepolia subgraph

* fix(OS-638): fixes typo in implementation

* chore: fix formatting in helpers script

* fix: ipfs returned an empty string (#485) (#495)

* chore: deploy 1.3.0-rc0 to arbitrum and arbitrumGoerli (#496)

* chore: prepared arbitrum deployment

* chore: deployed v1.3.0 to arbitrumGoerli

* docs: improved checklist

* chore: deployed v1.3.0 to arbitrum

* chore: deployed v1.3.0-rc0.4 npm packages

* chore: added manifests for arbitrum and arbitrumGoerli

* docs: improved deployment checklist

* docs: improved deployment checklist

* docs: fix typo

Co-authored-by: Mathias Scherer <[email protected]>

---------

Co-authored-by: Mathias Scherer <[email protected]>

* docs: v1.3.0 audit files

* release: deploy v1.3.0 (#509)

* docs(contracts): removes outdated step in upgrade checklist

* deploy(contracts): adds v1.3.0 mainnet deployment

* deploy(contracts): adds polygon deployment

* fix: removes uncessary inconsistent managingDAOImplementation field from active_contracts.json

* chore: bump package version to v1.3.0

* chore: updates version for osx package

* chore(contracts): updates changelog

* chore(contracts): updates active_contracts with implementations and bases

* chore(contracts): moves line in changelog

* docs: replace code4rena draft by final report

* docs: mention audit reports in README

* docs: added missing report link and use perma links

---------

Co-authored-by: Mathias Scherer <[email protected]>
heueristik added a commit that referenced this pull request Dec 20, 2023
* fix(subgraph): update to @graphprotocol/[email protected] (#453)

* fix: missing virtual keyword (#452)

* fix: added missing virtual keywords.

* fix: correct misspelled filename

* docs: improved NatSpec

* chore: bump package versions

* Update packages/contracts/CHANGELOG.md

Co-authored-by: Mathias Scherer <[email protected]>

---------

Co-authored-by: Mathias Scherer <[email protected]>

* Deploy contracts and subgraph to sepolia (#475)

* feat: deploy v1.3.0-rc0 to sepolia

* feat: bump subgraph version to v1.3.0-rc0
feat: deploy sepolia subgraph

* fix(OS-638): fixes typo in implementation

* chore: fix formatting in helpers script

* fix: ipfs returned an empty string (#485) (#495)

* chore: deploy 1.3.0-rc0 to arbitrum and arbitrumGoerli (#496)

* chore: prepared arbitrum deployment

* chore: deployed v1.3.0 to arbitrumGoerli

* docs: improved checklist

* chore: deployed v1.3.0 to arbitrum

* chore: deployed v1.3.0-rc0.4 npm packages

* chore: added manifests for arbitrum and arbitrumGoerli

* docs: improved deployment checklist

* docs: improved deployment checklist

* docs: fix typo

Co-authored-by: Mathias Scherer <[email protected]>

---------

Co-authored-by: Mathias Scherer <[email protected]>

* fix: build and release metadata where swapped

* fix: publish an empty JSON object to silence the EmptyReleaseMetadata error

* docs: improve checklists

---------

Co-authored-by: Mathias Scherer <[email protected]>
mathewmeconry added a commit that referenced this pull request Jan 4, 2024
* fix(subgraph): update to @graphprotocol/[email protected] (#453)

* fix: missing virtual keyword (#452)

* fix: added missing virtual keywords.

* fix: correct misspelled filename

* docs: improved NatSpec

* chore: bump package versions

* Update packages/contracts/CHANGELOG.md

Co-authored-by: Mathias Scherer <[email protected]>

---------

Co-authored-by: Mathias Scherer <[email protected]>

* Deploy contracts and subgraph to sepolia (#475)

* feat: deploy v1.3.0-rc0 to sepolia

* feat: bump subgraph version to v1.3.0-rc0
feat: deploy sepolia subgraph

* fix(OS-638): fixes typo in implementation

* chore: fix formatting in helpers script

* fix: ipfs returned an empty string (#485) (#495)

* chore: deploy 1.3.0-rc0 to arbitrum and arbitrumGoerli (#496)

* chore: prepared arbitrum deployment

* chore: deployed v1.3.0 to arbitrumGoerli

* docs: improved checklist

* chore: deployed v1.3.0 to arbitrum

* chore: deployed v1.3.0-rc0.4 npm packages

* chore: added manifests for arbitrum and arbitrumGoerli

* docs: improved deployment checklist

* docs: improved deployment checklist

* docs: fix typo

Co-authored-by: Mathias Scherer <[email protected]>

---------

Co-authored-by: Mathias Scherer <[email protected]>

* release: deploy v1.3.0 (#509)

* docs(contracts): removes outdated step in upgrade checklist

* deploy(contracts): adds v1.3.0 mainnet deployment

* deploy(contracts): adds polygon deployment

* fix: removes uncessary inconsistent managingDAOImplementation field from active_contracts.json

* chore: bump package version to v1.3.0

* chore: updates version for osx package

* chore(contracts): updates changelog

* chore(contracts): updates active_contracts with implementations and bases

* chore(contracts): moves line in changelog

* fix: build and release metadata mixups (#507)

* fix: build and release metadata where swapped

* fix: publish an empty JSON object to silence the EmptyReleaseMetadata error

* docs: improve checklists

* deploy: adds fqn to contract verification

* deploy: corrects false fqds

* chore: applies prettier formatting

* chore: adds arbitrum sepolia deployment

* deploy(subgraph): changes node endpoint to alchemy

* feat: deploys base sepolia
fix: sets fixed gas price for base networks
fix: fixes wrong multisig address for arbitrum sepolia management dao

* chore: adds release.md hr line

Co-authored-by: Michael Heuer <[email protected]>

* style: applies prettier formatting

---------

Co-authored-by: Michael Heuer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants