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.

Sign up for GitHub

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

Jump to bottom

[BUG] Not getting progressStatus on functions #299

Closed
DannyCastroVentura opened this issue Nov 24, 2022 · 0 comments · Fixed by #305
Closed

[BUG] Not getting progressStatus on functions #299

DannyCastroVentura opened this issue Nov 24, 2022 · 0 comments · Fixed by #305
Assignees
@DannyCastroVentura
Labels
bug Something isn't working

Comments

@DannyCastroVentura
Copy link
Contributor

Current Behavior

We are only getting Success or Failed progress status, not anything in between as we were having before.

Expected Behavior

See all the messages from "Starting script", to "initializing mcdevtools (for example)" and so on.

Steps To Reproduce

  1. Go to any 'userstory'
  2. Click on 'Refresh'
@DannyCastroVentura DannyCastroVentura added the bug Something isn't working label Nov 24, 2022
@DannyCastroVentura DannyCastroVentura self-assigned this Nov 24, 2022
DannyCastroVentura pushed a commit that referenced this issue Nov 24, 2022
#299 creating new apex method to return the current status, and start…
a402218 
… changing mcdo_retrieveTable in order to handle the new method
DannyCastroVentura pushed a commit that referenced this issue Nov 24, 2022
#299 adding the other functions, but still with the window.setTimeout…
857e7fa 
… error
DannyCastroVentura pushed a commit that referenced this issue Nov 24, 2022
#299 changed the from this.status to this.progressStatus
7dccdac
DannyCastroVentura pushed a commit that referenced this issue Nov 28, 2022
#299 change the mcdo_RunCopadoFunctionFromLWC.cls and mcdo_RetrieveTa…
638202b 
…ble.js to its previous state, once that this logic wont work, and will try to use the event's logic
DannyCastroVentura pushed a commit that referenced this issue Nov 28, 2022
#299 - added some console.logs
3b79faf
DannyCastroVentura pushed a commit that referenced this issue Nov 28, 2022
#299 - updated some console.logs
89cc167
DannyCastroVentura pushed a commit that referenced this issue Nov 29, 2022
#299 adding a method in order to get the job progress
eecf6c0
DannyCastroVentura pushed a commit that referenced this issue Nov 29, 2022
#299 using that method to retrieve the job progress on the message ca…
ed2284e 
…ll back
DannyCastroVentura pushed a commit that referenced this issue Nov 29, 2022
#299 changed the event that we are getting information about and chan…
b856c46 
…ge the logic as well so it can get the appropriate data, but it still have a bug, it is refreshing the table before the event actually ended
DannyCastroVentura pushed a commit that referenced this issue Nov 30, 2022
#299 changing logic to use both events, one to update the progress me…
7286b49 
…ssages, and the other to refresh the table once it the retrieve is done. New bug: sometimes the messages get confused with some other job steps (I think)
DannyCastroVentura pushed a commit that referenced this issue Nov 30, 2022
#299 fixing the bug, it wasn't unsubscribing, and so it was continuin…
4c774d0 
…g to get message from the older subscriptions as well
@DannyCastroVentura DannyCastroVentura linked a pull request Nov 30, 2022 that will close this issue
299 bug not getting progressstatus on functions #305
Merged
1 task
JoernBerkefeld added a commit that referenced this issue Dec 5, 2022
@JoernBerkefeld
#299: prototype of getResultIds
134ae1c 
still buggy; jobProgressWrapper & getJobProgress will be irrelevant when this works
DannyCastroVentura pushed a commit that referenced this issue Dec 5, 2022
#299 updating logic so we don't need to retrieve all the progress sta…
a80e128 
…tus from database but just the latest result id, and use the events to get it basing itself on the retrieved resultId
DannyCastroVentura pushed a commit that referenced this issue Dec 5, 2022
#299 changing logic so it gets all the result ids from the jobExcepti…
bf46ae6 
…onId
JoernBerkefeld added a commit that referenced this issue Dec 5, 2022
@JoernBerkefeld
#299: reduce result id response by filtering "closed" ones
290bce3
JoernBerkefeld added a commit that referenced this issue Dec 5, 2022
@JoernBerkefeld
#299: moved final Log.result behind uploading relevant JSON
163e581
JoernBerkefeld added a commit that referenced this issue Dec 5, 2022
@JoernBerkefeld
#299: remove custom "completed" listener
47367b7
This was referenced Dec 5, 2022
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DannyCastroVentura DannyCastroVentura

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

299 bug not getting progressstatus on functions Accenture/sfmc-devtools-copado
1 participant
@DannyCastroVentura