-
Notifications
You must be signed in to change notification settings - Fork 558
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
Realm Web: Basic structure and MongoDB Realm App types #2741
Conversation
Please rebase on |
|
50a21c7
to
016d279
Compare
016d279
to
2b39fc6
Compare
4ac9125
to
15e5c2b
Compare
I continued working from #2762 - so I think this is ready to be reviewed (and merged). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add license header to files.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall very nice, with some comments for changes
d592161
to
b18fc20
Compare
16796ab
to
5443902
Compare
5443902
to
791523f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
/** | ||
* Log out the currently authenticated user and clear any persisted authentication information. | ||
*/ | ||
logOut(): Promise<void>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't logOut
be on user only?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah - but I'll postpone that for phase 2, since this has already shipped before I learned about that change to the API.
/** | ||
* Pass an object implementing this interface to the app constructor. | ||
*/ | ||
interface AppConfiguration { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should also include name and version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you have a link to a description of those parameters? I didn't see them in any scopes but I do see them in the existing SDK.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added this a as a task on phase 2.
* Adding realm-web and realm-web-integration * Improving iterating integration tests * Implemented UserState & refactored transports * Using app-importer and running tests from node.js * Fixed BaseTransport issue * Using explicit devDependencies * Renamed FF to FunctionsFactoryType * Renaming Credentials#material to Credentials#toJSON() * Renamed "fetcher" to "transport" * Fixed a typo * App constructor now takes string or configuration * Moved from tslint to eslint * Moved "realm-web-integration" to "realm-web-integration-tests" * Changed from targeting es5 to es6 * Removed unneeded dependencies on ts-mocha * Fixed a test * Adding symbolic links to the LICENSE * Updated tests of the app * Incorporated feedback from review * Refactored "UsernamePasswordCredentials" into "EmailPasswordCredentials" * Renamed "Credentials" directory to "credentials" * Updated package locks * Updates and exports the base and default functions factories * Removed automatically bootstrapping on postinstall * Re-implemented Credentials * Updated the functions factory types * Adding the header eslint plugin * Adding headers to sourcefiles in ./packages
What, How & Why?
This closes the Realm Web part of #2699.
☑️ ToDos
Compatibility
label is updated or copied from previous entryBreaking
label has been applied or is not necessaryIf this PR adds or changes public API's: