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 aragonOS API #1

Closed
wants to merge 1 commit into from
Closed

Add aragonOS API #1

wants to merge 1 commit into from

Conversation

bingen
Copy link
Contributor

@bingen bingen commented May 15, 2018

Built with solidity-docgen.

@bingen bingen requested a review from luisivan May 15, 2018 13:09
Built with solidity-docgen.
"apm_APMRegistry_APMRegistryConstants",
"apm_Repo"
],
"APPS": [
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 make all of these not uppercase? Like Apps, Common, etc.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Well, those are generated by solidity-docgen. I can also not push those changes, or make another script to convert it to lowercase. i18n/en.json will need to changed too.

@@ -2,6 +2,9 @@
"scripts": {
"examples": "docusaurus-examples",
"sync-assets": "rsync -rtu \"$(dirname $(node -p 'require.resolve(\"@aragon/ui\")'))/\" static/aragon-ui",
"hack_solidity_version": "find ./node_modules/@aragon/os/contracts -name \"*.sol\" -exec sed -i 's/pragma solidity 0.4.18/pragma solidity ^0.4.18/g' {} \\;",
Copy link
Contributor

Choose a reason for hiding this comment

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

Instead of merging this PR with all the previously build files, would be cool to just merge the method of building them, and so we don't clutter the repo

@bingen
Copy link
Contributor Author

bingen commented May 16, 2018

Closing in favour of #2

@bingen bingen closed this May 16, 2018
0xGabi added a commit that referenced this pull request Dec 1, 2018
…n-on-js-ref

Format change of wrapper section on js-ref
unixpi referenced this pull request in unixpi/hack May 19, 2019
0xGabi pushed a commit that referenced this pull request May 22, 2019
* Update github-guide.md

Minor edits (tone + grammar)

* Add a couple of HQ images for testing

* Replace first three images

* Add a couple more hq images

* Replace remaining images in 'Editing a document on GitHub' subsection

* Add replacement images for 'Your first edit' subsection

* Delete gh-5-hq.png

Wrong folder

* Delete gh-6-hq.png

Wrong folder

* Add replacement images for 'Your first edit' subsection (correct folder)

* Edit 'Your first edit subsection' and link to replacement images

* Add replacement images for 'Proposing the file change' and 'Comparing changes' subsections

* Link to replacement images in 'Proposing the file change' and 'Comparing changes' subsections + minor edit

* Add replacement images for 'Your first pull request' subsection

* Add links to replacement images in 'Your first pull request' subsection + minor edits

* Add replacement images for 'Adjusting your pull request' subsection

* Add final links to replacement images + minor edits

* Halve width of zoomed in images

* Delete gh-0.png

* Delete gh-1.png

* Delete gh-10.png

* Delete gh-11.png

* Delete gh-12.png

* Delete gh-13.png

* Delete gh-14.png

* Delete gh-15.png

* Delete gh-16.png

* Delete gh-17.png

* Delete gh-18.png

* Delete gh-19.png

* Delete gh-2.png

* Delete gh-20.png

* Delete gh-3.png

* Delete gh-4.png

* Delete gh-5.png

* Delete gh-6.png

* Delete gh-7.png

* Delete gh-8.png

* Delete gh-9.png

* Center img
0xGabi pushed a commit that referenced this pull request Jun 1, 2019
@0xGabi 0xGabi deleted the aragon_os_api branch June 16, 2019 21:53
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