Comments (8)
Hello saif,
I would like to contribute in building this app. If you have any plan in mind for the app can you direct me towards it?
from marketplace-sample-apps.
That works @sarfaraz187 , anything as per your convenience.
from marketplace-sample-apps.
Hello @Saif-Shines,
Started working on Freshchat App Request and have implemented some of the features that had been requested. Kindly review the application and let me know your feedback Git Repo.
Here are few questions in which your suggestions can be helpful.
- Contact information displayed in modal is it sufficient?
- Since i am not any sending data back from modal to parent i haven't used instance API.
- Is it fine to display the last 5 lookups in app itself or should i display in modal?
from marketplace-sample-apps.
Hey @sarfaraz187,
Raise a PR and let's have it merged. Then we will focus on improvements gradually. 😄
from marketplace-sample-apps.
Hi @sarfaraz187 ,
This issue was created initially not for Hacktoberfest purpose, so this might require lot of time and effort. So you may choose to implement the aspects those will fit your time an expertise.
Having said that I will include it for hacktoberfest and let you know some of my thoughts that can help you as you plan to take it on.
- App can use the placeholder in Inbox Page.
- Use crayons to quickly build up a small form that takes in email, twitter_id and Fullname.
- Use Request Method to make an request and store last 5 responses in Data Store.
- This data got in response can be passed on to an Modal via Instance Method.
When this issue was created, Enrich API was free for developers to use to grab little details. I will let you choose any API that can be helpful that is free. The Goal is to demonstrate the features for others in the community to refer to. So having the part of just using Request Method is good enough without having to build complete app.
from marketplace-sample-apps.
Hello @Saif-Shines ,
Thanks for your suggestions and assigning me this App. Yes Saif i will keep in mind the time and effort required. As you have suggested will work on one module at a time.
Regarding the Enrich API,
- Had gone through the documentation and was planning to use Multi Field Request API.
- Enrich requires an API key in order to fetch data. Shall i setup an Iparams page to get the Enrich API key from the user. Let me know you thoughts on this?
from marketplace-sample-apps.
Hi @sarfaraz187 ,
We went through the code. Although there is a room for improvement, none of it is a blocker to have your code on this repository. If there is nothing else you'd add or subtract, please rase a PR adding app's code to Demo Apps folder.
Really appreciate your efforts and taking time to contribute.
from marketplace-sample-apps.
Hi @Saif-Shines,
Thanks for the update. If you have any suggestions for improvement do let me know or else if you had in mind for others in our community to contribute in the improvement of this app i'll be more than happy to raise PR request with the current version.
from marketplace-sample-apps.
Related Issues (20)
- [Bug] - Replace bootstrap as jQuery needs to be added for bootstrap
- [Bug] - Access token supplied contains Invalid chars [Demo-Apps/fullcontact_app]
- no error handler & descriptive help text
- [App Request] - config-placeholders-freshservice HOT 1
- [Sample App Request] - config-placeholders-crm
- [Sample App Request] - config-placeholders-freshchat
- [Sample App Request] - config-placeholders-freshteam
- [Bug] - freshhuddle.github.io no longer valid HOT 4
- [Bug] - Client db update add a "#" to the item key HOT 3
- [Feature Request] - Interface Method Sample App HOT 1
- [Enhancement] - Link sample apps to tutorials
- [Sample App] - Unit Tests for serverless parts
- [Bug] - Broken Links App Dev Features
- [Enhancement] - Refactor Ticket Merger
- [Enhancement] - Refactor Twilio SMS app
- [Bug] - how to refresh the accesstoken of freshdesk oauth app. HOT 3
- [Enhancement] - Move all the demo apps to individual repos in Freshworks developers org HOT 1
- [Bug] - For the external_events_app, register the serverless event handlers in manifest
- [Bug] - Crash Android FLAG_IMMUTABLE android 12 HOT 4
- How to call API without adding domain name in whitelisted HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from marketplace-sample-apps.