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 simple message popup #1704

Merged
merged 7 commits into from
Jun 8, 2020
Merged

add simple message popup #1704

merged 7 commits into from
Jun 8, 2020

Conversation

dkent600
Copy link
Contributor

@dkent600 dkent600 commented May 14, 2020

Replaces any need for alert. Motivated by the Biconomy PR.

DaoLandingPage is now using it, thus simplifying the code in that component.

DAOCreator should be made to use it:
dOrgTech/DAOcreator#502 (comment)

@dkent600 dkent600 changed the title simpleMessagePopup add simple message popup May 14, 2020
@dkent600 dkent600 had a problem deploying to alchemy-simplemessagepo-dciqaq May 14, 2020 10:23 Failure
@dkent600 dkent600 temporarily deployed to alchemy-simplemessagepo-dciqaq May 14, 2020 10:27 Inactive
@dkent600 dkent600 temporarily deployed to alchemy-simplemessagepo-jvkhpc May 14, 2020 11:12 Inactive
@dkent600 dkent600 temporarily deployed to alchemy-simplemessagepo-jvkhpc May 14, 2020 11:36 Inactive
@dkent600 dkent600 requested a review from orenyodfat May 14, 2020 11:37
@dkent600 dkent600 self-assigned this May 14, 2020
Copy link
Contributor

@orenyodfat orenyodfat left a comment

Choose a reason for hiding this comment

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

Too much code change for a simple message popup .
unless there are new linting rules in the pr ,which I do not see ones, please update only the code needed and avoid unneeded , not related to this pr changes..so it will be easy to review .

@dkent600
Copy link
Contributor Author

dkent600 commented May 25, 2020

Regarding linting, the only changes that I see involve whitespace, which you can eliminate yourself from the diff using the github GUI in the "Files Changed" tab of the PR:

image

@dkent600 dkent600 force-pushed the simpleMessagePopup branch from 6b3f279 to 11354e7 Compare June 4, 2020 18:11
@dkent600 dkent600 temporarily deployed to alchemy-simplemessagepo-jvkhpc June 4, 2020 18:11 Inactive
@dkent600 dkent600 temporarily deployed to alchemy-simplemessagepo-jvkhpc June 4, 2020 18:21 Inactive
@dkent600 dkent600 merged commit e8e9d39 into dev Jun 8, 2020
@dkent600 dkent600 deleted the simpleMessagePopup branch June 8, 2020 16:04
dkent600 added a commit that referenced this pull request Jul 10, 2020
* show only register schemes in pluging manager (#1661)

* show only register schemes in pluging manager

* remove subscription

* Can't vote if didn't have reputation in DAO when proposal was created (#1634)

* Can't vote if didn't have reputation in DAO when proposal was created

Fixes #1593
Depends on daostack/arc.js#451

* use arc.js 0.2.68

* update package-lock.json

* @daostack/arc.js

* documentation

* fix gh-issue #1655

* Update CHANGELOG.md

* fix changelog

* add comment

* tweaks to comment

* fix votingDisabled

Co-authored-by: Oren Sokolowsky <[email protected]>
Co-authored-by: orenyodfat <[email protected]>
Co-authored-by: Doug Kent <[email protected]>

* Update subgraph_endpoints.json (#1662)

* add console warn if scheme has future activation time (#1669)

* Update schemeUtils.ts

* lint

* xDAI: Remove burner connect , Add Tor.us (#1652)

* Update settings.ts

xDAI:
- Remove burner connect (Not in use)
- add Tor.us

* Update settings.ts

* Update settings.ts

* update settings.ts

Updated with the xDAI network URL

* Update settings.ts

* Update settings.ts

* Update settings.ts

* Update settings.ts

* Update settings.ts

* Update settings.ts

* web3modal v 1.6.3

* spelling

* update package-lock.json

* add network to torus..option

* setting.

* lint

Co-authored-by: orenyodfat <[email protected]>
Co-authored-by: Oren Sokolowsky <[email protected]>

* temorary workaround for gh issue #1684 (#1691)

* add plugin to SchemeRegstrar plugins list (#1688)

* breadcrumbs for dao landing page, proposal plugins (#1683)

* refactor and fix appearance of New Proposal button (#1681)

* Bump version and update changelog (#1694)

* bump version, update changelog

* edit

* remove references to alchemy server

* update nodejs version in readme

* updating daocreator versionm (#1697)

* add help to proposal create description boxes (#1687)

* better embedded video UX (#1685)

* Fix plugin manage edit "plugin manager" scheme . permission (#1717)

* Fix plugin manage edit "plugin manager" scheme . permission

* travis Increasing the amount of inotify watchers

* comment

* remove first docker-compose log alchemy -travis

* remove docker log alchemy from travis

* cosmetics

* fix ensPublicResolver setAddr encoding (#1718)

* fix Disqus scoping (#1702)

* first try

* fix disqusConfig initialization

* fix new proposal button (#1713)

* show notification on copy to clipboard (#1724)

* show notification on copy to clipboard

* simplify code

* fix edit, don't show new/edit unless known (#1723)

* set torus z-index (#1727)

* changelog and bump version for 0.10.8 (#1721)

* bump version to 0.10.8

* changelog

* more stuff added

* typo

* warn when GenericScheme `value` is > 0 (#1733)

* show genericscheme value, in red when > 0

* change wording

* blinking warning, and on the card too

* handle unknown details

* handle DutchX details

* update selenium chrome version

* embed videos everywhere (#1707)

* create ProposalDescription component

* add to feed, competitions and submissions

* help text for competition+submission description

* add orderby (#1740)

* fix stake amount GEN label (#1743)

* fix layout

* remove redundant style

* subscribe to redm. changes, tooltip when disabled (#1746)

* sosCollective Landing Page (#1714)

* customized SoS Collective landing page

* comment

* fix onboarding URL

* load voter profiles in VotersModal (#1730)

* load voter profiles in VotersModal

* don't getProfile unless needed

* default user account when creating CR proposal (#1708)

* let current user be default

* show current account as placeholder

* fix redemption numbers in menu (#1749)

* url params for Competitions/Submissions (#1712)

* generate url

* associate createSubmission modal with url

* add EnsPublicResolver address (#1762)

* release-2020-03-11 (#1494)

* Close modals by pressing ESC

* Fix styling of submit button on profile page when submitting

* Make sure tooltip on redemptions button doesn't cover the button

* Create ModalPopup component to bundle common modal stuff

Share common modal styles across all modals.
All ModalPopups can be closed by pressing esc or clicking on the background
All modals have a header, body and optional footer

* Fix paths

* In create proposal form tab directly from title to description

Fixes: #964

* Lint fix

* in history, open proposal in new tab on ctrl-click (#1474)

* open proposal in new tab with ctrl-click

* typo

* handle mac command key

* show DAO total rep in sidebar (#1473)

* Enforce required permissions when adding/editing known Schemes

Also move all scheme utilities into a new lib/schemeUtils.ts file

* Require NETWORK env var on Heroku and default to rinkeby

* Add generic scheme json

* Add ens registrar

* Preserve ts file formatting

* overflow generic scheme fix

* init

* init

* Bounty network integration (#1102)

* added preliminary json

* added preliminary json

* added contract addresses, changed contract name

* new react component

* update fields mass

* revert uint fields

* contract name

* Update ProposalSummaryStandardBounties.tsx

* Finish StandardBounties.json

* Update ProposalSummaryStandardBounties.tsx

* Update ProposalSummaryStandardBounties.tsx

* Conditionally render Bounties tab

* Update SchemeContainer.tsx

* Begin adding open bounties tab

* Prepare component for open bounties

* Generate List of Open Bounties

* parse markdown bounty details

* add Known Scheme entry

* update addresses

* Update StandardBounties.json

* fix open bounties display

* fixed CSS for larger form generic schemes and rearranged generic scheme action order for Standard Bounties

* Bugfix, forms of type number would return error on trim() function, as it applies only to strings

* Update StandardBounties.json

* Fix Bounties Navigation Tabs

* Fix proposals for Bounties and set ETH as default bounty token

* Add tests

* Update fields of test

* Specified on kind of timestamp for deadline field

* Changed unit to Gwei, add conditional for bounties API

* Update ProposalSummaryStandardBounties.tsx

* Update ProposalSummaryStandardBounties.tsx

* Add bounty ID to API, return bounty stage filter

* Clarified user input for API compatibility

* Return issuer in Bounties API call

* Edit typo

* Fix error due to typo

* fix sent eth amounts

* Change details to data, per naming convention

* Update SchemeOpenBountyPage.tsx

* Lint and isolate ethToSend for Standard Bounties

* Update proposal-genericSchemeStandardBounties.ts

* Comment out tests

* Update Mainnet Address to latest current

* As by code review

* Lint, Remove JSON decimals for fields, convert relevant fields to Wei avoiding precision errors

* Removed log used in testing

* added transformation to Bounties JSON scheme

* updated statement to ECMA 2016 spec

* Updated tests for Bounties

* check test for standard bounties

* revert test

* compilation fixes

Co-authored-by: Edson Ayllon <[email protected]>
Co-authored-by: Edson Ayllon <[email protected]>

Co-authored-by: Thomas Spofford <[email protected]>
Co-authored-by: Tibet Sprague <[email protected]>
Co-authored-by: Tibet Sprague <[email protected]>
Co-authored-by: dOrgJelli <[email protected]>
Co-authored-by: Alex Zak <[email protected]>
Co-authored-by: Edson Ayllon <[email protected]>
Co-authored-by: Edson Ayllon <[email protected]>

* Revert "release-2020-03-11 (#1494)"

This reverts commit d5c9b86.

* add EnsPublicResolver address

Co-authored-by: Jelle <[email protected]>
Co-authored-by: Thomas Spofford <[email protected]>
Co-authored-by: Tibet Sprague <[email protected]>
Co-authored-by: Tibet Sprague <[email protected]>
Co-authored-by: dOrgJelli <[email protected]>
Co-authored-by: Alex Zak <[email protected]>
Co-authored-by: Edson Ayllon <[email protected]>
Co-authored-by: Edson Ayllon <[email protected]>

* fix count of days in countdowns (#1764)

* fix count of days in countdowns

* single version of calculateCountdown

* fix All Daos page layout problems (#1761)

* fix All Daos page layout problems

* cleaner mid-width

* clean up sidebar on mobile

* fix historic display of voting percentages (#1780)

* validateDaoProposalRoute (#1768)

* validate is the current DAO's proposal

* reference DAO name in message

* give dao landing page padding on mobile (#1778)

* remove redundant New Proposal button (#1783)

* line up votingDisabled text (#1782)

* augment mintBurnReputation permission description (#1769)

* augment mintBurnReputation permission description

* same fix on scheme info

* changelog and bump version to 0.10.9 (#1759)

* changelog and bump version to 0.10.9

* more changes

* updates

* added Mint and Burn  plugin permission

* break out array, and truncate long param values (#1799)

* break out array, and truncate long param values

* improved readability

* more improved readability

* more on readability

* remove spaces

* init

* comment

* improve sidebar scroll, fix DAOstack icon color (#1793)

* close Redemptions menu on Redeem click (#1803)

* improve layout of AccountProfile popup (#1772)

* add simple message popup (#1704)

* added SimpleMessagePopup

* convert to redux state

* lint

* make accessible from utils.ts

* lint

* fix div styling

* ensure  init is the same as the rest of the app

* removed all cases of dash at end of infinitescroll (#1804)

* improve scheme info layout (#1787)

* replace redundant copy to clipboards ops with component (#1806)

* replace redundant copy to clip ops with component

* merge error

* page the preboosted proposals (#1755)

* page the preboosted proposals

* cheap-source-map

* fix merge errors

* use contract alias for plugin name (#1790)

* use contract alias for plugin name

* update package-lock

* use contract alias for plugin name

* update package-lock

* rename test

* swap EnsPublicResolver and EnsRegistry addresses

* cheap-source-map

* don't create pretty alias

* don't prettify rewarder contract alias

* revert tests

* bump version, update changelog for 0.10.10 (#1815)

* bump version, update changelog

* new features

* don't split GenericScheme name (#1826)

* Add dxswap governance scheme (#1816)

* added dxswap scheme

* added dxswap scheme

* Daocreator xdai support (#1791)

* updating daocreator version to support xdai

* Change of devtool to 'cheap-source-map'

* fix proposal count not updating (#1817)

* navigator.webdriver => global.inAlchemyTests (#1814)

* Update issue templates

* Create config.yml

* Update config.yml

* Update bug_report.md (#1839)

* handle wide GenericScheme calldata (#1857)

* truncate call data

* back to miniscule

* BUIDLHub proposal notifications integration changes (#1843)

* BUIDLHub proposal notifications integration changes

* Added buidlhub assets

* Fixed popup message modal to use hideFooter vs. custom button spec

* Fixed linter problems

* Changed h2 to inline-block

* v0.10.11 changelog (#1874)

* bump version (#1877)

* fix copy to clipboard for callData array params (#1883)

* I added the GEMS token to the list which is needed for i4 (#1821)

* fix insufficient GEN error message layout (#1801)

* clean up PreTransactionModal on mobile (#1777)

* custom landing page for the Venus Project (#1878)

* Custom data for Venus Project

* make network-specific

* instructions in the readme

* update readme

* more in readme

* clean up voting and staking UI (#1872)

* fix xDAI issues (#1913)

* send to blockscout in address/tx links

* RPC URLs

* use getArc()

* update provider URL

* poll for eth changes

* changelog, bump version to 12 (#1920)

* fix compile errors

* lint

* remove redundant CSS change

* upgraded arc.js, compile succeeds

* update package-lock

* upgrade test-env to match arc.js

* fix broken test

* fix bug in custom landing page on ganache

* update changelog

* edit Alchemy Goods

* rename Alchemy goods

* further documentation scheme=>plugin changes

* test bug fixes

* more test fixes

* genesisprotocol voteOnBehalf address can be null

* list error

* remove scss.d files

* fix proposal description field errors

* use targetedNetwork

* remove .ps1 from .gitignore

* add *.css.d.ts to .gitignore

Co-authored-by: orenyodfat <[email protected]>
Co-authored-by: Tibet Sprague <[email protected]>
Co-authored-by: Oren Sokolowsky <[email protected]>
Co-authored-by: Theeylon <[email protected]>
Co-authored-by: Cesar Brazon <[email protected]>
Co-authored-by: Jelle <[email protected]>
Co-authored-by: Thomas Spofford <[email protected]>
Co-authored-by: Tibet Sprague <[email protected]>
Co-authored-by: dOrgJelli <[email protected]>
Co-authored-by: Alex Zak <[email protected]>
Co-authored-by: Edson Ayllon <[email protected]>
Co-authored-by: Edson Ayllon <[email protected]>
Co-authored-by: Jordan Ellis <[email protected]>
Co-authored-by: Nico Elzer <[email protected]>
Co-authored-by: Michael Coon <[email protected]>
Co-authored-by: arsena21 <[email protected]>
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