You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@devlake.apache.org by GitBox <gi...@apache.org> on 2022/08/26 12:49:19 UTC
[GitHub] [incubator-devlake] e2corporation opened a new pull request, #2859: feat: setup database migration alert dialog on ui
e2corporation opened a new pull request, #2859:
URL: https://github.com/apache/incubator-devlake/pull/2859
### ⚠️ Config-UI / App / Database Migration Alert
> WIP DO NOT MERGE
- [ ] **Feature** Add Database Migration Alert Dialog & UX Flow
- [ ] **Refactor** Remove UI/Rendering obligations from `Request.js` XHR Utility
- [ ] Allow User-controlled Migration Confirmation Request
- [ ] Test Migration Routine
### Description
This PR adds a formal Database Migration Alert Dialog to Configuration UI. If the API has detected a migration, all it's resource responses will return a `428` Status response, in which case a special `localStorage` key will be set to indicate that a migration warning is in effect. The Alert is mounted at the Application level so that it renders no matter what Page Service is being accessed.
### Does this close any open issues?
#2788
### Screenshots
`[<PENDING>]`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-devlake] klesh merged pull request #2859: feat: setup database migration alert dialog on ui
Posted by GitBox <gi...@apache.org>.
klesh merged PR #2859:
URL: https://github.com/apache/incubator-devlake/pull/2859
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-devlake] e2corporation commented on pull request #2859: feat: setup database migration alert dialog on ui
Posted by GitBox <gi...@apache.org>.
e2corporation commented on PR #2859:
URL: https://github.com/apache/incubator-devlake/pull/2859#issuecomment-1235026720
@klesh I would like to get this merged to `main` soon so the feature can be tested early, thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-devlake] e2corporation commented on pull request #2859: feat: setup database migration alert dialog on ui
Posted by GitBox <gi...@apache.org>.
e2corporation commented on PR #2859:
URL: https://github.com/apache/incubator-devlake/pull/2859#issuecomment-1234262136
Rebased
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-devlake] e2corporation commented on pull request #2859: feat: setup database migration alert dialog on ui
Posted by GitBox <gi...@apache.org>.
e2corporation commented on PR #2859:
URL: https://github.com/apache/incubator-devlake/pull/2859#issuecomment-1233157765
Rebased
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org