[12.0] web_notify: improve popup UI #1231
Merged
Conversation
This file contains 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
simahawk
changed the title
simahawk
requested changes
Apr 2, 2019
|
@simahawk Did everything as you requested ;) |
|
@shepilov-vladislav thanks! One thins tho: seems demo is not working on runbot, if I hit the buttons nothing happens (beside refreshing the form) |
It's required longpoling. Longpoling works on runbot? |
…ssage_received` to `on_message` for prevent collisions.
@simahawk Can you double-check? I made some edits. |
|
@shepilov-vladislav on runbot it does not work but I think you are right about longpolling. |
simahawk
approved these changes
Apr 5, 2019
tschanzt
pushed a commit
to camptocamp/web
that referenced
this pull request
Jun 19, 2019
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
Laurent-Corron
pushed a commit
to acsone/web
that referenced
this pull request
Nov 14, 2019
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
kevinkhao
pushed a commit
to akretion/web
that referenced
this pull request
Nov 5, 2020
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
dzungtran89
pushed a commit
to dzungtran89/web
that referenced
this pull request
Feb 16, 2021
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
BankyKmitl
pushed a commit
to BankyKmitl/web
that referenced
this pull request
Dec 13, 2021
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
em230418
pushed a commit
to em230418/web
that referenced
this pull request
Mar 31, 2022
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
em230418
pushed a commit
to em230418/web
that referenced
this pull request
Apr 28, 2022
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
aiendry-aktivsoftware
pushed a commit
to aiendry-aktivsoftware/web
that referenced
this pull request
Aug 10, 2022
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
aiendry-aktivsoftware
pushed a commit
to aiendry-aktivsoftware/web
that referenced
this pull request
Aug 15, 2022
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
JordiBForgeFlow
pushed a commit
to ForgeFlow/web
that referenced
this pull request
Sep 15, 2022
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
gh-aswanth
pushed a commit
to gh-aswanth/OCA-web
that referenced
this pull request
Oct 15, 2022
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
gh-aswanth
pushed a commit
to gh-aswanth/OCA-web
that referenced
this pull request
Oct 15, 2022
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
chienandalu
pushed a commit
to Tecnativa/web
that referenced
this pull request
Jan 5, 2023
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
baimont
pushed a commit
to acsone/web
that referenced
this pull request
Feb 17, 2023
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
baimont
pushed a commit
to acsone/web
that referenced
this pull request
Apr 11, 2023
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
Nikul-OSI
pushed a commit
to ursais/web
that referenced
this pull request
Feb 13, 2024
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
Nikul-OSI
pushed a commit
to ursais/web
that referenced
this pull request
Feb 16, 2024
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
trisdoan
pushed a commit
to trisdoan/web
that referenced
this pull request
Oct 3, 2024
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
trisdoan
pushed a commit
to trisdoan/web
that referenced
this pull request
Oct 7, 2024
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
trisdoan
pushed a commit
to trisdoan/web
that referenced
this pull request
Oct 7, 2024
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
trisdoan
pushed a commit
to trisdoan/web
that referenced
this pull request
Oct 14, 2024
* [ADD]: all available bootstrap notifications (success/danger/warning/info/default) * [IMP] use black color for text for default notification. * [FIX] reverted require string for `bus.Longpolling` and rename `on_message_received` to `on_message` to prevent collisions.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Added all available bootstrap-like notifications (success/danger/warning/info/default).