search

CrumbleThorn / CS-197-ACR-ICD

The PhilHealth ACR ICD Code Database integrated onto the East Avenue Medical Center Excel Spreadsheet.
MIT License
2 stars 4 forks source link

Excel 2016 issue #4

Open roycan opened 7 years ago

roycan commented 7 years ago

Both Excel 2016 and Office 365 doesn't allow RVS codes checkbox to be checked.

Excel 2016 is not yet popular, but it will be in a few years, so we best fix it or plan a bit for it

chrstnnglc commented 7 years ago

Hi Sir, we've found out that not all macros are supported by the newer version :)

roycan commented 7 years ago

Interesting. AFAIK Microsoft hasn’t broken backward compatibility since the dawn of .NET … Maybe another big change for microsoft is coming. Are any of the unsupported functionality crucial to what we want to do? Things like this are expected to happen, I just want it to happen around 2 years from now, or at least not on the same month we came out with the tool. Office 365 is not significantly used. Office 2016 will probably go mainstream, at least for those getting new laptops.

Anyway, the team’s mission remains the same:
To provide ICD/ RVS look up within the Excel Sheets used by Census-assigned Doctors, who will deem the software useful and valuable.

I don’t know if my UVLe message reached team leader Gab. If you can, please let him know. Perhaps we should have scheduled another code-review session following the last set of changes we did. I will be in UP today and tomorrow this week, as well as Thursday-Saturday next week should anyone wish to consult.

Thank you for reminding me about this Christine! Things would have been a lot worse, if I presented it to the doctors any nearer to the grades submission date.

Good luck to your team. Let’s turn this 1.0-turned-5.0 to a 1.0 !

-S’R

On 26 May 2017, at 9:34 AM, Christine Felizardo notifications@github.com wrote:

Hi Sir, we've found out that not all macros are supported by the newer version :)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/CrumbleThorn/CS-197-ACR-ICD/issues/4#issuecomment-304167782, or mute the thread https://github.com/notifications/unsubscribe-auth/AE0uBvouBO48I7Zo6eYMYdcL_q1XgAShks5r9iw4gaJpZM4NmhO3.

sharleengo commented 7 years ago

Sir, I found out that the issue with the check boxes came from the code that I added to make sure that at least one of the two check boxes (ICD and RVS) was checked so that a search won't be made when neither one is checked. However, this code is causing errors in 365 and 2016 so I decided to get rid of it. Currently, the two check boxes are now independent of one another and they may both be unchecked. I think this is rather a small issue compared to the first issue when the macros wouldn't work on 365 and 2016. So currently, it works in 365 with the small issue mentioned above.

roycan commented 7 years ago

Hi Sharleen,

That’s very good news. I shall try to request a presentation on Saturday. Make sure the new code is committed to Christine’s github. I’ll update my fork from there.

Happy lunch,

On 26 May 2017, at 11:18 AM, sharleengo notifications@github.com wrote:

Sir, I found out that the issue with the check boxes came from the code that I added to make sure that at least one of the two check boxes (ICD and RVS) was checked so that a search won't be made when neither one is checked. However, this code is causing errors in 365 and 2016 so I decided to get rid of it. Currently, the two check boxes are now independent of one another and they may both be unchecked. I think this is rather a small issue compared to the first issue when the macros wouldn't work on 365 and 2016. So currently, it works in 365 with the small issue mentioned above.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/CrumbleThorn/CS-197-ACR-ICD/issues/4#issuecomment-304180559, or mute the thread https://github.com/notifications/unsubscribe-auth/AE0uBnVNE0Ln8laenopybu7ib9FI03o1ks5r9kRqgaJpZM4NmhO3.