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

Release/0.2.0 #123

Merged
merged 14 commits into from
Nov 18, 2022
Merged

Release/0.2.0 #123

merged 14 commits into from
Nov 18, 2022

Conversation

makylfang
Copy link
Contributor

PR

Describe your changes

  • Release v0.2.0

Checklist before requesting a review

  • You have performed a self-review of your own code
  • You are using approved terminology
  • Your changes have been tested locally
  • Your code builds clean without any errors or warnings

shaloo and others added 13 commits November 3, 2022 09:56
* Removed code related to handling processing of request

* Initial commit for plug n play auth

* Added closeFunc to ModalParams, replaced useState with useReducer in modal.tsx, updated action component with method

* Fixed form button to prevent default routing, Added connect button in examples, fixed twitter icon link, fixed login list for connect to be fetched

* Added loader svg, added css variables

* Moved theme to a different file, updated styles to be generated from theme

* Updated versions for babel-jest and jsdom

* Fixed import in components.tsx for getTheme

* merged dev, updated classnames to have xar- prefix, updated closing modal to throw error

* Getting logo and theme from authprovider

* Added style.css for modal, added postcss and rollup module for it

* removed theme.ts, added stroke for loader in css file

* Hotfix: Refs: #AR-3986 usage guide

* typo fix Update usage.md

* bug fix Update usage.md

* Updated example for connect

* Using fallback from utils

* Removed sourcemap generation

* Removed theme from ui/typings, imported from base/typings, removed circular dependency bug

* Updated build target to commonjs, removed a circular dependency bug

* Updated version

* Removed max- min- width with width, added box-sizing to content-box

* Updated z-index, added font-family to header subtext

* Added EthereumProvider interface, Updated version, ts emits declaration only, added type as module in package.json

Co-authored-by: shaloo <5890484+shaloo@users.noreply.github.com>
Co-authored-by: shalz <shalz@hotmail.com>
* Added showWidgetBubble function to be called on connect, hidden by default

* Updated version, added back `build:ts` in `build` script
* Removed params from .init(), added to constructor params, removed no-ui mode, widget or full mode chosen by alwaysShowWidget flag - defaults to true

* Removed sentry, removed polyfill dependencies, renamed errors to start with Error..., taking theme as user input

* Updated documentation

* Updated error names
* Updated alwaysShowWidget flag to alwaysVisible

* Removed InitParams since its not used now, removed related function, stopped exporting AppMode from main file

* Update version
* Update position to fixed and zIndex to maximum value

* Reduce z-index to 100000
@netlify
Copy link

netlify bot commented Nov 18, 2022

Deploy Preview for authsdk-ref-guide ready!

Name Link
🔨 Latest commit 7d99cb6
🔍 Latest deploy log https://app.netlify.com/sites/authsdk-ref-guide/deploys/63775c648b5112000889f06c
😎 Deploy Preview https://deploy-preview-123--authsdk-ref-guide.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Comment on lines +168 to +176
// console.log('Doing encryption')
// setRequest('encrypt')
// const c = await encryptWithPublicKey({
// publicKey,
// message: plaintext,
// })
// setResult(c)
// console.log({ ciphertext: c })
// ciphertext = c

Choose a reason for hiding this comment

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

Some commented out code here and the example index.html. Was it intentional?

Copy link
Contributor Author

@makylfang makylfang Nov 18, 2022

Choose a reason for hiding this comment

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

Yeah since encryption got removed from SDK, wasn't possible to encrypt in example.

@makylfang makylfang merged commit 7d99cb6 into main Nov 18, 2022
@makylfang makylfang deleted the release/0.2.0 branch November 18, 2022 10:27
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.

None yet

4 participants