Skip to content

Conversation

@rafzk
Copy link
Contributor

@rafzk rafzk commented Oct 25, 2021

Creating diagram using GoJS library

rafzk added 21 commits October 8, 2021 16:19
Using 'Attachment API - POST /now/attachment/file' to upload multiple files on form submit.
This widget is using the 'Attachment API - POST /now/attachment/file' to upload multiple files on form submit.

The result will appear in the console or check directly the record.
In this snippet you will create a custom process flow with a custom page and populated with standard widgets.
This reverts commit 22fbfc3.
Record process flow
This change will correct an issue which prevent window users to clone this repo.

Error
error: invalid path 'UI Actions/Add "Show Workflow" Related link/readme.md'
Creating diagram using GoJS library
@rafzk
Copy link
Contributor Author

rafzk commented Oct 25, 2021

image

Copy link
Contributor

@smartcarleen smartcarleen left a comment

Choose a reason for hiding this comment

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

Hi @rafzk! This is pretty cool! When I implemented it in my PDI I found a discrepancy. Your readme says:

Client Side Scripts

  • Copy and Paste below Server-Side Script in Widget's Server Side Section

Can you correct this to say that it needs to go inside the Client controller function?

Also, there was a note in your CSS code:

// need to improve this section

I'm not sure if you meant to fix this before you submitted it or do you plan to come back later with enhancements?

@rafzk
Copy link
Contributor Author

rafzk commented Oct 26, 2021

Yes sure @smartcarleen! I've updated the readme 👍

@rafzk
Copy link
Contributor Author

rafzk commented Oct 26, 2021

Yah ServiceNow is using ChartJS but its always good to explore other alternatives 🥇 I will do one using ChartJS so people can compare it

Copy link
Contributor

@smartcarleen smartcarleen left a comment

Choose a reason for hiding this comment

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

Hi @rafzk Please also correct the part of your readme that directs the user to paste the client-side code into the Server-side script. It should say something like:

  • Copy and Paste inside the api.controller function in the Client Controller section

I tried to make these simple changes myself, but I don't have access.

@rafzk
Copy link
Contributor Author

rafzk commented Oct 26, 2021

Done. Can you check @smartcarleen ?

@rafzk
Copy link
Contributor Author

rafzk commented Oct 26, 2021

Yah, you cant! The original code its on my repo and all changes needs to be done on my end unless you accept it and change it later. Thank you!!

Copy link
Contributor

@smartcarleen smartcarleen left a comment

Choose a reason for hiding this comment

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

Thanks for making those changes! Happy Hacktoberfest!

@smartcarleen smartcarleen merged commit 82f2d5e into ServiceNowDevProgram:main Oct 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants