upavadi / TngApi

Current Version, 3.3.3 released on 29 July 2022. The TngApi plugin for Wordpress is a stand-alone plugin. It integrates smoothly with TNG to display genealogy data in Wordpress pages
http://www.upavadi.net
0 stars 2 forks source link

family form #24

Closed tboulogne closed 10 years ago

tboulogne commented 10 years ago

Hello,

I successfully setup the tng api. But i get a problem on submiting update family form. 1/ Get a 404 on the thank-you page 2/ Data not updated

Could you explain more how to make this part working please ?

Thanks for help.

Regards,

Thierry

upavadi commented 10 years ago

Hi Thierry, Pleased to note that the API installed correctly. My mistake. I forgot to mention that you would need to create a wordpress page named 'thank-you' with your own Thank You message for the submitter. Hopefull ythat should resolve the issue.

tboulogne commented 10 years ago

Thanks.

I create this page but data are not updated. Any tips ?

Thierry BOULOGNE

Le 16 août 2014 à 23:28, upavadi notifications@github.com a écrit :

Hi Thierry, Pleased to note that the API installed correctly. My mistake. I forgot to mention that you would need to create a wordpress page named 'thank-you' with your own Thank You message for the submitter. Hopefull ythat should resolve the issue.

— Reply to this email directly or view it on GitHub.

upavadi commented 10 years ago

An email is sent to the administrator with details of changes, when user Adds or Update family data. At the moment there is no facility to update this data, directly. Did you receive email with changes - ( Email would not work if you are on local server.)

tboulogne commented 10 years ago

Hello,

Thanks for your answer. Could you explain exactly what is the blocking part to update database ? You get access to database and all datas (id, familly,.....)

I quickly lookup the database, it's seems classical.

Let me know we will look together to improve this :-) (we could see how TNG do, and redo same :-) )

upavadi mailto:notifications@github.com 17 août 2014 00:19

An email is sent to the administrator with details of changes, when user Adds or Update family data. At the moment there is no facility to update this data, directly. Did you receive email with changes - ( Email would not work if you are on local server.

— Reply to this email directly or view it on GitHub https://github.com/upavadi/TngApi/issues/24#issuecomment-52407437.

TNG CONCEPTS - Formation - Prestations Informatiques - Distribution http://www.tng-concepts.com

Mettre la Technologie au service de l'Homme. Votre exigence. Notre Métier. Thierry BOULOGNE Responsable développement (+33) 5 35 00 53 90 (Fixe) (+33) 6 43 18 31 19 (Mob.)

Formation - E-commerce - informatique www.tng-concepts.com http://www.tng-concepts.com

tboulogne commented 10 years ago

Hello,

I look into the code and make a try with updating data into database. look TngContent file attached (from line 628).

I identify some interesting files into TNG directory with needed functions for the update. We could share if you're interesting to make this modifications.

I play with zebra_datepicker to add some datepicker for easy date entry too. Not in the file, but very simple to implement;

Let me if you want we proceed together.

Regards,

upavadi mailto:notifications@github.com 17 août 2014 00:19

An email is sent to the administrator with details of changes, when user Adds or Update family data. At the moment there is no facility to update this data, directly. Did you receive email with changes - ( Email would not work if you are on local server.

— Reply to this email directly or view it on GitHub https://github.com/upavadi/TngApi/issues/24#issuecomment-52407437.

upavadi commented 10 years ago

Updating data in the database is on our to-do list. (issues 5 & 6). Personally, I am not comfortable with users updating my database. Best way to achieve this is to store changes in a temporary table. The administrator would check the changes. If OK, administrator can transfer changes from temporary table to TNG tables. You are most welcome to fork and implement your own changes. I would be very interested in your ideas.

tboulogne commented 10 years ago

Hello,

No need to get a temporary database. TNG get a reviewing process depending on user right access. So could study how it works and apply. In this state, without database integration, the form don't have a lot of interest => 2 data entries with potentials error on the second entry by admin.

So if you don't want direct data update, the data could be format in a way for easy database integration : => csv file for DB import => gedcom format for TNG import

The goal is to avoid several data entries.

upavadi mailto:notifications@github.com 18 août 2014 10:41

Updating data in the database is on our to-do list. (issues 5 & 6). Personally, I am not comfortable with users updating my database. Best way to achieve this is to store changes in a temporary table. The administrator would check the changes. If OK, administrator can transfer changes from temporary table to TNG tables. You are most welcome to fork and implement your own changes. I would be very interested in your ideas.

— Reply to this email directly or view it on GitHub https://github.com/upavadi/TngApi/issues/24#issuecomment-52465589.

TNG CONCEPTS - Formation - Prestations Informatiques - Distribution http://www.tng-concepts.com

Mettre la Technologie au service de l'Homme. Votre exigence. Notre Métier. Thierry BOULOGNE Responsable développement (+33) 5 35 00 53 90 (Fixe) (+33) 6 43 18 31 19 (Mob.)

Formation - E-commerce - informatique www.tng-concepts.com http://www.tng-concepts.com

upavadi commented 10 years ago

I think you misunderstood my intention. The idea is to give the user flexibility in updating data. The administrator would check data before updating database. Best way to achieve this is to allow the administrator to see the data submitted in one table and data in the database, in another table. We may be able to highlight changes submitted. Once administrator has made any changes ( like correcting date format), a 1-click transfer can take place. To achieve this, 1 - Create a temporary table in wordpress to hold user submitted data. ( The idea here is not to mess with TNG database ) 2 - To check, Administrator can look at data in TNG and data in user submitted table and decide whether the changes submitted are appropriate. 3 - If all ok, adminstrator, can transfer all data from wordpress table to appropriate TNG tables.

There is no manual double data entry. I hope that makes sense.

tboulogne commented 10 years ago

Hello,

Thanks for your reply. My concern was on current version of the api : was so surprise on the mail data sending. No problem on pushing data into a temporary database. Need a fluent process for admin to check quiclky what data change as github with commit. Checking two interfaces could be hard.

For information, i push some modifications in my fork on error handling when customer is not logged. I will push in next day, integration of zebra_datepicker for date entry beginning by years (need to improve this).

upavadi mailto:notifications@github.com 19 août 2014 10:06

I think you misunderstood my intention. The idea is to give the user flexibility in updating data. The administrator would check data before updating database. Best way to achieve this is to allow the administrator to see the data submitted in one table and data in the database, in another table. We may be able to highlight changes submitted. Once administrator has made any changes ( like correcting date format), a 1-click transfer can take place. To achieve this, 1 - Create a temporary table in wordpress to hold user submitted data. ( The idea here is not to mess with TNG database ) 2 - To check, Administrator can look at data in TNG and data in user submitted table and decide whether the changes submitted are appropriate. 3 - If all ok, adminstrator, can transfer all data from wordpress table to appropriate TNG tables.

There is no manual double data entry. I hope that makes sense.

— Reply to this email directly or view it on GitHub https://github.com/upavadi/TngApi/issues/24#issuecomment-52602236.

TNG CONCEPTS - Formation - Prestations Informatiques - Distribution http://www.tng-concepts.com

Mettre la Technologie au service de l'Homme. Votre exigence. Notre Métier. Thierry BOULOGNE Responsable développement (+33) 5 35 00 53 90 (Fixe) (+33) 6 43 18 31 19 (Mob.)

Formation - E-commerce - informatique www.tng-concepts.com http://www.tng-concepts.com

upavadi commented 10 years ago

Hi Thank you for your continued interest. I would like you to be aware that

Date Picker: Jquery date picker was modified to show month/year only. The day defaults to first day of the format. Date Format used in this plug-in is dd/mm/yyyy The reason for this is 1 - TNG database uses this convention 2 - we, in UK, use this convention So it makes sense to stick to this convention. Hope that helps.

tboulogne commented 10 years ago

Datepicker :

Presenting year before is ergonomical interest for year like 1900. Less click to find. Généalogical format in tng is dd M yy as mention in documentation. Get 2 fields in database One with dd M yy And one with yyyy-dd-MM. Any date entry is concerted to genealogical format (v10). For french peuple better date is genealogical format, so i Will use this.

Thierry BOULOGNE

Le 20 août 2014 à 10:12, upavadi notifications@github.com a écrit :

Hi Thank you for your continued interest. I would like you to be aware that

We developed this plug-in for personal use and have been using it for about 9 months there is no commercial interest. There were requests from other TNG users to make code available. Hence it is on Github It was designed with 1-data security, 2 - data integrity 3- user experience 4- admin functions Priority in that order So putting data in the temporary table is low on our priority list. Date Picker: Jquery date picker was modified to show month/year only. The day defaults to first day of the format. Date Format used in this plug-in is dd/mm/yyyy The reason for this is 1 - TNG database uses this convention 2 - we, in UK, use this convention So it makes sense to stick to this convention. Hope that helps.

— Reply to this email directly or view it on GitHub.