-
Notifications
You must be signed in to change notification settings - Fork 2
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
Migrate snekclientv2 aa1 to implement 1 #116
Conversation
Now the search uses a new task which delivery only user pages. Therefore filtering is no longer needed. Previously there was a plain username string list which got replaced by a object list containing slugs and titles.
The argument has been updated to fit the new structure.
The login does no longer need a environment file including the anonymous username. The anonymous status can be be obtained directly from the authentication object.
The package has been downgraded to 11.0.7 due to issues with the latest version. Ref: mosch/react-avatar-editor#340
The package.json has been updated with the beta release of v2 SnekClient.
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.
Should be ok to merge after a few changes.
Now when whoami is undefined the condition will not pass.
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.
LGTM
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.
LGTM
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.
Should be ok to merge.
Checklist:
Type of Changes:
What is the current behavior? (link to any open issues here)
What is the new behavior (if this is a feature change)?
Other information: