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

incorrect menu + do not merge dev #43

Open
matthewberryman opened this issue Aug 6, 2018 · 0 comments
Open

incorrect menu + do not merge dev #43

matthewberryman opened this issue Aug 6, 2018 · 0 comments

Comments

@matthewberryman
Copy link
Contributor

Hi. Just noting there's a bug in dev in which after you login (now using universal auth0 thanks to @mehrdadgit's work) it still has the pre-login menu (with option to login "Masuk", rather than options for map "Peta" and logout "Keluar"). Neither of us know our way around aurelia routing well enough (yet) in order to fix it, but in light of other discussions around a rewrite of REM (and using Cognito) it doesn't seem worth fixing this bug. Also just flagging that dev now should not be merged into master as we've gone with two different auth0 ways of doing the login and merging dev -> master would break prod without some corresponding config changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant