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

ui: Bugfixes #571

Merged
merged 4 commits into from
Oct 22, 2020
Merged

ui: Bugfixes #571

merged 4 commits into from
Oct 22, 2020

Conversation

daniel-arnauer
Copy link
Contributor

@daniel-arnauer daniel-arnauer commented Oct 21, 2020

Closes #554
Closes #566
Closes #555
Closes #558
Closes #557

@daniel-arnauer daniel-arnauer self-assigned this Oct 21, 2020
@daniel-arnauer daniel-arnauer changed the title Fix disable user himself small Bugfixes Oct 21, 2020
@daniel-arnauer
Copy link
Contributor Author

daniel-arnauer commented Oct 21, 2020

Here is the excel sheet with information about the translation changes and what I implemented or rejected

French_Translation_Suggestions_commented.xlsx

@openkfwCI
Copy link

openkfwCI commented Oct 21, 2020

NotesTime
Note for Reviewer: E2E tests on remote server succeededThu, 22 Oct 2020 16:28:05 +0000

Generated by E2E-Test

Comment on lines 190 to 193
const mapDispatchToProps = dispatch => {
return {
fetchGroups: () => dispatch(fetchGroups(true)),
fetchGroups: () => dispatch(fetchGroups(false)),
fetchProjectPermissions: pId => dispatch(fetchProjectPermissions(pId, false)),
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All fetch functions must not show a loading indicator to fix #555

@daniel-arnauer daniel-arnauer force-pushed the small-bugfixes branch 3 times, most recently from 15a0bdf to 5793ec7 Compare October 22, 2020 15:03
@daniel-arnauer daniel-arnauer marked this pull request as ready for review October 22, 2020 15:04
daniel-arnauer added 4 commits October 22, 2020 17:56
Fix double indicator in Confirmation Dialog[#555]
Show loading indicator while uploading a document [#558]
Added and fixed some translation [566]
Fix stlye buf in GroupDialog [#557]
Remove <;> at LoginPage
Fix: To prevent a user from disabling himself, the button
to disable the current logged-in user has been hidden. [#554]
@Stezido Stezido changed the title small Bugfixes ui: Bugfixes Oct 22, 2020
@Stezido Stezido merged commit fced586 into master Oct 22, 2020
@Stezido Stezido deleted the small-bugfixes branch October 27, 2020 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants