-
Notifications
You must be signed in to change notification settings - Fork 41
Refactor JS #241
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
Open
rafaella-martino
wants to merge
35
commits into
master
Choose a base branch
from
rm--refactor_js
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Refactor JS #241
Changes from 1 commit
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
ac998c2
feat: remove credential.js
rafaella-martino 1532925
feat: remove ajax, rename controllers actions, use js alerts to displ…
rafaella-martino cb11999
feat: remove `message.js` as it is no longer used
rafaella-martino 521b0c2
feat: remove unused js targets
rafaella-martino 7bddc67
feat: replace `fetch` for `submit`, change `credential` param name to…
rafaella-martino 871ab25
test: update test with renamed params
rafaella-martino cbf503e
feat: change `.then` for `await` for readability
rafaella-martino a990e01
feat: add permitted params in controllers
rafaella-martino bd72e31
feat: pass url as values, add token to session js controller
rafaella-martino 6115812
feat: merge js controllers into one
rafaella-martino e19ef24
feat: change submit buttons, rename js controller variables
rafaella-martino 9b46e85
fix: attend deprecation warning
rafaella-martino 6fe61ca
Merge branch 'master' into rm--refactor_js
rafaella-martino 6002921
style: use rails button helpers
rafaella-martino e93e3e8
feat: use `expect` to require params
rafaella-martino 95b5664
feat: change `create_options` and `get_options` controller actions for
rafaella-martino 331859e
fix: add space in css
rafaella-martino c42c9fc
rollback: use messenger.js to display error messages
rafaella-martino 7a35c2c
feat: use `requestjs` gem
rafaella-martino 81ad153
fix: Display layout instead of json when Webauthn Verification error
rafaella-martino c2753ed
feat: use render instead of redirect when request fails, use flash for
rafaella-martino 1c92613
feat: Change variable names in JS controller
rafaella-martino 5307cb1
refactor: move `request.js` import with the rest of the imports
rafaella-martino 449c9cb
refactor: temporarily use `redirect` when `create` credential fails,
rafaella-martino e8b7ea3
refactor: enabling `submit` button when clicking `Cancel` with js
rafaella-martino 9e58703
feat: clear not saved credential if there is an error
rafaella-martino 672bcb1
Merge branch 'master' into rm--refactor_js
rafaella-martino b9edbe2
make rubocop happy
rafaella-martino 23a316e
refactor: remove unnecessary cleanup when error updating credential
rafaella-martino 55e10a6
rollback handling Webauthn Error exception
rafaella-martino e35488d
refactor: rollback to use json responses in case of errors
rafaella-martino 3accfb1
refactor: leave controller response for next PR, rollback to respond
rafaella-martino 18b76a5
change response message if registration success
rafaella-martino 72e5f40
test: add new controller changes
rafaella-martino 82a766f
stop displaying message
rafaella-martino File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
is it okay to show the error if there's no message?
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.
Maybe we could add a generic error message? Like the one we set before:
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.
Sounds good!